[OpenWrt-Devel] [RFT][PATCH] toolchain/musl: update to 1.2.0

Rosen Penev rosenp at gmail.com
Sun Mar 22 05:37:26 EDT 2020


On Sun, Mar 22, 2020 at 2:25 AM Hannu Nyman <hannu.nyman at welho.com> wrote:
>
> Rosen Penev wrote at Sat Mar 21 20:36:15 PDT 2020:
>
>  > Remove GLOB_ONLYDIR patch. Does not seem to be needed.
>  > ...
>  > The GLOB_ONLYDIR macro is only needed for fstools, which should be  fixed
> there.
>
>
> Which of those conflicting statements is true?
Probably both. I did not test absolutely every package. Some fail to
compile, even on the buildbots. That's before this patch btw. I also
made the latter comment after compiling most of the packages feed.
>
> "GLOB_ONLYDIR does not seem to be needed" or
> "The GLOB_ONLYDIR macro is only needed for fstools" so this patch would in
> effect break fstools unless it is modified ???
>
> The first one is in the proposed commit message, while the second is in the
> comments section.
>
>
> Grepping the source directories of my normal build for wrt3200acm I think
> that fstools still needs it:
I will submit a patch for fstools when I can.
>
> perus at ub1910:/Openwrt/wrt3200/build_dir/target-arm_cortex-a9+vfpv3_musl_eabi$
> grep -r GLOB_ONLYDIR .
>    ...
>    /fstools-2020-01-21-deb745f8/libfstools/overlay.c:    if (!glob(globdir,
> GLOB_NOESCAPE | GLOB_MARK | GLOB_ONLYDIR, NULL, &gl))
>
>
> (My own build has also some other packages that seem to to have GLOB_ONLYDIR
> in their source, but both nano and gnulib in gdb define it by themselves.)
And that's exactly the proper behavior. fstools is bugged and must be patched.
>
> I would be cautious in removing the patch with just "Does not seem to be
> needed" unless the need is really evaluated with all feeds.
As stated above, packages that don't define this are bugged and must be patched.
>
>
> In any case, great to see a patch for musl 1.2.0 version bump.
> Would be good to have that in before the next release branching.
>
>
> ps. The commit message has misspelled "woresize.h". Btw, how did you evaluate
> that wordsize.h is not needed any more?
I compiled most of the packages feed and base feeds. These includes
are hacks to be honest. They shouldn't be here. Unfortunately, too
many packages break without them currently.
>
>

_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list