| Message ID | 20200425061531.12845-1-carenas@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | macos: do not assume brew and gettext are always available/wanted | expand |
Hi Carlo, On Fri, 24 Apr 2020, Carlo Marcelo Arenas Belón wrote: > since 27a7067308 (macos: do let the build find the gettext > headers/libraries/msgfmt, 2020-04-23) a build with `make NO_GETTEXT=1` > will throw warnings like : > > LINK git > ld: warning: directory not found for option '-L/usr/local/opt/gettext/lib' > > localize the change together with all the other package specific tweaks > and make sure it only applies when both gettext was needed and brew was > the provider. Okay, that makes sense, but... > > Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> > --- > Makefile | 9 +++++++++ > config.mak.uname | 7 ++----- ... moving this into the `Makefile`, i.e. make this even more independent from Homebrew than before, is probably a bad idea. I agree with Eric Sunshine who complained that my patch is not dependent on the use of Homebrew. I haven't found a way to make it more dependent (there is no `NO_HOMEBREW` knob, even though there are `NO_FINK` and `NO_DARWIN_PORTS`), but that does not mean that we should make it even worse. Will update my patch to guard the options a bit better. Ciao, Dscho > 2 files changed, 11 insertions(+), 5 deletions(-) > > diff --git a/Makefile b/Makefile > index 9804a0758b..031a231ad6 100644 > --- a/Makefile > +++ b/Makefile > @@ -1303,6 +1303,15 @@ ifeq ($(uname_S),Darwin) > BASIC_LDFLAGS += -L/opt/local/lib > endif > endif > + ifndef NO_GETTEXT > + ifeq ($(shell test -d /usr/local/opt/gettext/ && echo y),y) > + BASIC_CFLAGS += -I/usr/local/opt/gettext/include > + BASIC_LDFLAGS += -L/usr/local/opt/gettext/lib > + ifeq ($(shell test -x /usr/local/opt/gettext/bin/msgfmt && echo y),y) > + MSGFMT = /usr/local/opt/gettext/bin/msgfmt > + endif > + endif > + endif > ifndef NO_APPLE_COMMON_CRYPTO > NO_OPENSSL = YesPlease > APPLE_COMMON_CRYPTO = YesPlease > diff --git a/config.mak.uname b/config.mak.uname > index 540d124d2e..0ab8e00938 100644 > --- a/config.mak.uname > +++ b/config.mak.uname > @@ -133,11 +133,8 @@ ifeq ($(uname_S),Darwin) > HAVE_BSD_SYSCTL = YesPlease > FREAD_READS_DIRECTORIES = UnfortunatelyYes > HAVE_NS_GET_EXECUTABLE_PATH = YesPlease > - BASIC_CFLAGS += -I/usr/local/include -I/usr/local/opt/gettext/include > - BASIC_LDFLAGS += -L/usr/local/lib -L/usr/local/opt/gettext/lib > - ifeq ($(shell test -x /usr/local/opt/gettext/bin/msgfmt && echo y),y) > - MSGFMT = /usr/local/opt/gettext/bin/msgfmt > - endif > + BASIC_CFLAGS += -I/usr/local/include > + BASIC_LDFLAGS += -L/usr/local/lib > endif > ifeq ($(uname_S),SunOS) > NEEDS_SOCKET = YesPlease > > base-commit: 27a706730868835ec02a21a9ac4c4fcb3e05d330 > -- > 2.26.2.569.g1d74ac4d14 > > >
diff --git a/Makefile b/Makefile index 9804a0758b..031a231ad6 100644 --- a/Makefile +++ b/Makefile @@ -1303,6 +1303,15 @@ ifeq ($(uname_S),Darwin) BASIC_LDFLAGS += -L/opt/local/lib endif endif + ifndef NO_GETTEXT + ifeq ($(shell test -d /usr/local/opt/gettext/ && echo y),y) + BASIC_CFLAGS += -I/usr/local/opt/gettext/include + BASIC_LDFLAGS += -L/usr/local/opt/gettext/lib + ifeq ($(shell test -x /usr/local/opt/gettext/bin/msgfmt && echo y),y) + MSGFMT = /usr/local/opt/gettext/bin/msgfmt + endif + endif + endif ifndef NO_APPLE_COMMON_CRYPTO NO_OPENSSL = YesPlease APPLE_COMMON_CRYPTO = YesPlease diff --git a/config.mak.uname b/config.mak.uname index 540d124d2e..0ab8e00938 100644 --- a/config.mak.uname +++ b/config.mak.uname @@ -133,11 +133,8 @@ ifeq ($(uname_S),Darwin) HAVE_BSD_SYSCTL = YesPlease FREAD_READS_DIRECTORIES = UnfortunatelyYes HAVE_NS_GET_EXECUTABLE_PATH = YesPlease - BASIC_CFLAGS += -I/usr/local/include -I/usr/local/opt/gettext/include - BASIC_LDFLAGS += -L/usr/local/lib -L/usr/local/opt/gettext/lib - ifeq ($(shell test -x /usr/local/opt/gettext/bin/msgfmt && echo y),y) - MSGFMT = /usr/local/opt/gettext/bin/msgfmt - endif + BASIC_CFLAGS += -I/usr/local/include + BASIC_LDFLAGS += -L/usr/local/lib endif ifeq ($(uname_S),SunOS) NEEDS_SOCKET = YesPlease
since 27a7067308 (macos: do let the build find the gettext headers/libraries/msgfmt, 2020-04-23) a build with `make NO_GETTEXT=1` will throw warnings like : LINK git ld: warning: directory not found for option '-L/usr/local/opt/gettext/lib' localize the change together with all the other package specific tweaks and make sure it only applies when both gettext was needed and brew was the provider. Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> --- Makefile | 9 +++++++++ config.mak.uname | 7 ++----- 2 files changed, 11 insertions(+), 5 deletions(-) base-commit: 27a706730868835ec02a21a9ac4c4fcb3e05d330