Re: css.c: No such file or directory when lex lib not found during build

2023-05-24 Thread Darshit Shah
Hi,

Thanks for reporting this. I think we can modify the configure script to make 
the check on flex a hard error.

I'll do that soon(tm)


On Wed, May 24, 2023, at 11:01, David Cepelik wrote:
> Hi all,
>
> I noticed some strange behavior when building recent wget (fbbdf9ea)
> from sources using recent Flex (d30bdf4) built from sources. I tracked
> the issue down to Flex [1], but I thought that wget's toolchain made the
> problem harder to debug, so I figured it might be a good think to report
> it here as well.
>
> In a nutshell, when lex library is not detected, configure will output
> the following:
>
> [...]
> checking for flex... flex
> checking for lex output file root... lex.yy
> checking for lex library... not found
> configure: WARNING: required lex library not found; giving up on flex
> [...]
>
> which gets drowned in the rest of the output. Even though the library is
> "required" as per the warning above, the build process continues and
> eventually crashes with,
>
> [...]
> echo '#include "wget.h"' > css_.c
> cat css.c >> css_.c
> cat: css.c: No such file or directory
> make[3]: *** [Makefile:3156: css_.c] Error 1
> [...]
>
> This is because LEX defaults to : in src/Makefile:
>
> [...]
>   CC   convert.o
>   CC   cookies.o
> :  -ocss.c css.l
>   CC   ftp.o
>   CC   css-url.o
> [...]
>
> Is this expected?
>
> Best, David
>
> [1] https://github.com/westes/flex/issues/565
> Attachments:
> * signature.asc



css.c: No such file or directory when lex lib not found during build

2023-05-24 Thread David Cepelik
Hi all,

I noticed some strange behavior when building recent wget (fbbdf9ea)
from sources using recent Flex (d30bdf4) built from sources. I tracked
the issue down to Flex [1], but I thought that wget's toolchain made the
problem harder to debug, so I figured it might be a good think to report
it here as well.

In a nutshell, when lex library is not detected, configure will output
the following:

[...]
checking for flex... flex
checking for lex output file root... lex.yy
checking for lex library... not found
configure: WARNING: required lex library not found; giving up on flex
[...]

which gets drowned in the rest of the output. Even though the library is
"required" as per the warning above, the build process continues and
eventually crashes with,

[...]
echo '#include "wget.h"' > css_.c
cat css.c >> css_.c
cat: css.c: No such file or directory
make[3]: *** [Makefile:3156: css_.c] Error 1
[...]

This is because LEX defaults to : in src/Makefile:

[...]
  CC   convert.o
  CC   cookies.o
:  -ocss.c css.l
  CC   ftp.o
  CC   css-url.o
[...]

Is this expected?

Best, David

[1] https://github.com/westes/flex/issues/565

signature.asc
Description: signature