Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

crypto_botan: botan 2.13 header deprecation warnings #3385

Closed
mpranj opened this issue Apr 9, 2020 · 3 comments
Closed

crypto_botan: botan 2.13 header deprecation warnings #3385

mpranj opened this issue Apr 9, 2020 · 3 comments
Assignees
Labels
build dependencies Pull requests that update a dependency file

Comments

@mpranj
Copy link
Member

mpranj commented Apr 9, 2020

As of botan 2.13.x (maybe earlier?) we started getting the following header deprecation warnings:

In file included from ../src/plugins/crypto/botan_operations.cpp:12:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/hmac.h:14:1: warning: this header will be made internal in the future [-W#pragma-messages]
BOTAN_FUTURE_INTERNAL_HEADER(hmac.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:107:49: note: expanded from macro 'BOTAN_FUTURE_INTERNAL_HEADER'
      #define BOTAN_FUTURE_INTERNAL_HEADER(hdr) _Pragma("message \"this header will be made internal in the future\"")
                                                ^
<scratch space>:100:2: note: expanded from here
 message "this header will be made internal in the future"
 ^
In file included from ../src/plugins/crypto/botan_operations.cpp:13:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/init.h:16:1: warning: this header is deprecated [-W#pragma-messages]
BOTAN_DEPRECATED_HEADER(init.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:104:42: note: expanded from macro 'BOTAN_DEPRECATED_HEADER'
    #define BOTAN_DEPRECATED_HEADER(hdr) _Pragma("message \"this header is deprecated\"")
                                         ^
<scratch space>:102:2: note: expanded from here
 message "this header is deprecated"
 ^
In file included from ../src/plugins/crypto/botan_operations.cpp:14:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/lookup.h:35:1: warning: this header is deprecated [-W#pragma-messages]
BOTAN_DEPRECATED_HEADER(lookup.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:104:42: note: expanded from macro 'BOTAN_DEPRECATED_HEADER'
    #define BOTAN_DEPRECATED_HEADER(hdr) _Pragma("message \"this header is deprecated\"")
                                         ^
<scratch space>:104:2: note: expanded from here
 message "this header is deprecated"
 ^
In file included from ../src/plugins/crypto/botan_operations.cpp:15:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/pbkdf2.h:16:1: warning: this header will be made internal in the future [-W#pragma-messages]
BOTAN_FUTURE_INTERNAL_HEADER(pbkdf2.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:107:49: note: expanded from macro 'BOTAN_FUTURE_INTERNAL_HEADER'
      #define BOTAN_FUTURE_INTERNAL_HEADER(hdr) _Pragma("message \"this header will be made internal in the future\"")
                                                ^
<scratch space>:106:2: note: expanded from here
 message "this header will be made internal in the future"
 ^
In file included from ../src/plugins/crypto/botan_operations.cpp:17:
In file included from /usr/local/Cellar/botan/2.13.0/include/botan-2/botan/sha2_32.h:12:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/mdx_hash.h:13:1: warning: this header will be made internal in the future [-W#pragma-messages]
BOTAN_FUTURE_INTERNAL_HEADER(mdx_hash.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:107:49: note: expanded from macro 'BOTAN_FUTURE_INTERNAL_HEADER'
      #define BOTAN_FUTURE_INTERNAL_HEADER(hdr) _Pragma("message \"this header will be made internal in the future\"")
                                                ^
<scratch space>:108:2: note: expanded from here
 message "this header will be made internal in the future"
 ^
In file included from ../src/plugins/crypto/botan_operations.cpp:17:
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/sha2_32.h:14:1: warning: this header will be made internal in the future [-W#pragma-messages]
BOTAN_FUTURE_INTERNAL_HEADER(sha2_32.h)
^
/usr/local/Cellar/botan/2.13.0/include/botan-2/botan/compiler.h:107:49: note: expanded from macro 'BOTAN_FUTURE_INTERNAL_HEADER'
      #define BOTAN_FUTURE_INTERNAL_HEADER(hdr) _Pragma("message \"this header will be made internal in the future\"")
                                                ^
<scratch space>:110:2: note: expanded from here
 message "this header will be made internal in the future"
 ^
6 warnings generated.

Botan maintains a docu of deprecated headers.

@mpranj mpranj added build dependencies Pull requests that update a dependency file labels Apr 9, 2020
@markus2330
Copy link
Contributor

Thank you for reporting this problem!

@petermax2: what is the best way forward? Maybe we should remove some of the crypto plugins (with duplicated functionality) to keep maintenance effort reasonable?

@petermax2
Copy link
Member

what is the best way forward? Maybe we should remove some of the crypto plugins (with duplicated functionality) to keep maintenance effort reasonable?

Since the variations of the crypto plugin have been introduced for academic research (whis has been finished two years ago) I think the best way to go on is to keep the libgcrypt version and drop the rest (botan, OpenSSL). I will open another ticket.

@petermax2
Copy link
Member

Botan support was removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

3 participants