-
Notifications
You must be signed in to change notification settings - Fork 442
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
Added NEWS and README with links to md files #346
Conversation
NEWS and README are mandatory for autoconf. Fixes problem introduced in b1a81b3
Wouldn't it be wiser to add |
Seems to be an other way to solve the issue, but I'm not the one to decide... |
I saw the build failure because of my commit(s). Sorry for not checking the makefile for changes. |
No problem. Who is in charge for deciding the actual resolution? @neomilium @smortex @doegox ? |
If @smortex solution works, I'm more in favor of it that creating phony files. |
NEWS and README are mandatory for autoconf. Fixes problem introduced in b1a81b3 closes nfc-tools/libnfc#346 fixes nfc-tools/libnfc#347
done and fixed in #349 |
The applied fix in #349 is insufficient as this still fails for cmake. CMake Error at /usr/share/cmake-3.6/Modules/CPack.cmake:423 (message): @frankmorgner how does @hiddewie compile libnfc? As both automake and cmake complain(ed) loudly ... |
NEWS and README are mandatory for autoconf. Fixes problem introduced in
b1a81b3
The current status can be checked here https://travis-ci.org/frankmorgner/vsmartcard/jobs/122279341#L2087