-
Notifications
You must be signed in to change notification settings - Fork 28
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
Error: possibly undefined macro: m4_esyscmd #100
Comments
I have the same issue |
I have the same issue, see #101 (comment) Also, there's the solution with cmake in the comment. I guess it'd be good to add CMake to the README @kevinkreiser in case the autoconf route is failing, I'll do a quick sketch.. |
Using the alternative cmake build, I ran into an issue with missing headers on MacOS Big Sur. @kevinkreiser pointed me at the CI build script which specifies
Opened #105 with an update to the readme. |
sorry for not offering up this find sooner, it for some reason didnt occur to me that these were the same issue 🤦♂️ |
I tried running that but got this error: Do you or @kevinkreiser happen to know why? Thank you! |
@ntq2503 I think you have a typo in the cmake argument, |
the not-working autotools way annoyed the life out of me.. I found a way it's working for my arch linux:
(from arch linux forum) No idea if that's fixing mac installations too. Would you mind giving it a shot? |
@nilsnolde we might be able to modify |
just had a look at |
@kevinkreiser after a dozen git pulls, I still don't know what the hell is going wrong.. with the following
it's still giving the same error in the It does produce not-quite-right looking libs, but the symlinking doesn't care (
No matter in what sequence (or which ones I left out) I ran the above commands with Maybe you wanna try the above |
I do not have #!/bin/bash
aclocal -I m4
autoheader --warnings=no-portability
autoconf --warnings=no-portability
automake --force-missing --add-missing fixes it for me. |
@stari4ek nice! I'll see if that works on other platforms and get it updated if so |
did you ever get around to try my suggestion @kevinkreiser ? for the github actions mac runner this fails on 10.15 catalina (also, as you know, the brew one is not working atm). guess that'll sooner or later become a problem on circle ci as well, couldn't find out what mac os version they're using, I guess < 10.15. there's still the cmake build of course. is there a particular reason why you don't want to entirely switch to cmake? |
no i havent.. since i recently did a bunch of CI stuff for https://github.com/vthiery/cpp-statsd-client/ i think i might just convert this project to using github actions. then ill run into the problem as you point out and ill be forced to do it, what do you t hink? i am really liking github actions! |
Haha sure, sounds good. |
i realize i didnt fully answer your question
circle ci for valhalla was broken when using brew but i fixed it by building prime_server from source instead of using brew at all. for macos i dont care about using cmake vs autotools but for linux i absolutely stand by autotools as it plays very nicely with debian packaging. that is one of the main things i regret about cmake for valhalla. i would have liked to have continued to do ppa packages for valhalla but maintaining such a large library in both autotools and cmake doesnt make sense. instead i should update the cmake to make both shared and static libraries, apparently it is possible.. and then figure out how to update my debian packaging tooling to work with cmake. the first time i learned to do that it took a hell of a lot of work though so i have just kind of been relying on docker as a means of "releasing" public binaries. frankly i kind of hate it though.. maybe i should just go to ubuntu snaps instead of ppa but frankly i feel like ppa still beats snaps in terms of usage |
Right, I knew I missed smth.. always forget the packaging stuff.. cmake packaging sounds very painful 😅 or maybe Debian packaging is.. |
Hi! I'm wondering if the fix for this will be included in any release anytime soon? |
@julianskartor since this was a build tooling issue i had never considered it important enough to release but I can do so if you mean github release |
@kevinkreiser I see. After a closer inspection of the commits on top of 0.7.0, it seems there's no API/functional changes? Since this problem only affects my local development I can probably just use specific commit hash instead 👍 |
Hi,
I'm trying to install on macOS Big Sur 11.2.3 and ran into this issue after running ./autogen.sh.
All of my autotools are installed and up-to-date:
autoconf 2.71
automake 1.16.3_1
zeromq 4.3.4
czmq 4.2.0
Error message:
configure.ac:1: warning: AC_INIT: unsafe as a filename: "prime_server-[m4_esyscmd([./version.sh])]"
configure.ac:23: warning: The preprocessor macro `STDC_HEADERS' is obsolete.
configure.ac:23: Except in unusual embedded environments, you can safely include all
configure.ac:23: ISO C90 headers unconditionally.
configure.ac:41: warning: $as_echo is obsolete; use AS_ECHO(["message"]) instead
lib/m4sugar/m4sh.m4:692: _AS_IF_ELSE is expanded from...
lib/m4sugar/m4sh.m4:699: AS_IF is expanded from...
./lib/autoconf/general.m4:2249: AC_CACHE_VAL is expanded from...
./lib/autoconf/general.m4:2270: AC_CACHE_CHECK is expanded from...
m4/ax_pthread.m4:88: AX_PTHREAD is expanded from...
configure.ac:41: the top level
configure.ac:1: error: possibly undefined macro: m4_esyscmd
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
autoreconf: error: /usr/local/Cellar/autoconf/2.71/bin/autoconf failed with exit status: 1
I've tried looking everywhere online but haven't found a solution to this. Please advise!
Thank you so much!
The text was updated successfully, but these errors were encountered: