Re: [Toybox] Errors While Compling fold and/or defconfig: xgetdelim not found
github CI green again and android's build is green (haven't tested yet; i'll let https://android-review.googlesource.com/c/platform/external/toybox/+/2768464 run the tests for me :-) ). On Mon, Oct 2, 2023 at 1:07 PM Rob Landley wrote: > > On 10/2/23 10:16, enh via Toybox wrote: > > yeah, this broke the github CI too --- > > https://github.com/landley/toybox/actions/runs/6369602289/job/17289987237 > > Oops. > > Try now, > > Rob > ___ > Toybox mailing list > Toybox@lists.landley.net > http://lists.landley.net/listinfo.cgi/toybox-landley.net ___ Toybox mailing list Toybox@lists.landley.net http://lists.landley.net/listinfo.cgi/toybox-landley.net
Re: [Toybox] Errors While Compling fold and/or defconfig: xgetdelim not found
On 10/2/23 10:16, enh via Toybox wrote: > yeah, this broke the github CI too --- > https://github.com/landley/toybox/actions/runs/6369602289/job/17289987237 Oops. Try now, Rob ___ Toybox mailing list Toybox@lists.landley.net http://lists.landley.net/listinfo.cgi/toybox-landley.net
Re: [Toybox] Errors While Compling fold and/or defconfig: xgetdelim not found
yeah, this broke the github CI too --- https://github.com/landley/toybox/actions/runs/6369602289/job/17289987237 On Sun, Oct 1, 2023 at 10:52 PM Oliver Webb via Toybox wrote: > > When I saw that fold got promoted, I pulled the repo and ran make. Instead of > compiling cleanly, fold.c gave a error that a 'xgetdelim()' didn't exist. > "grep -R"-ing through the code, it was only mentioned in fold.c. > > I tried to fix this originally by trying to implement xgetdelim, but > the problem is probably that some un-commited changes were made to > lib/xwrap.c. which fold.c based itself on. then when _only_ fold.c got > committed were errors apparent. > > I can't get a good fix for this command that looks reasonable, compiles, > and passes it's tests; Best I have done was make something that passed the > first test case. > > Rob, If you have some changes you made to xwrap.c. Can you commit them? > "make defconfig toybox" no longer works. And everything about this error makes > me think this was a "Develop and test fold.c under a uncommited version of > xwrap.c with xgetdelim(), then only update fold.c" issue. Especially since > versions of fold before it's promotion don't use xgetdelim and compile ceanly > > - Oliver Webb > > ___ > Toybox mailing list > Toybox@lists.landley.net > http://lists.landley.net/listinfo.cgi/toybox-landley.net ___ Toybox mailing list Toybox@lists.landley.net http://lists.landley.net/listinfo.cgi/toybox-landley.net
[Toybox] Errors While Compling fold and/or defconfig: xgetdelim not found
When I saw that fold got promoted, I pulled the repo and ran make. Instead of compiling cleanly, fold.c gave a error that a 'xgetdelim()' didn't exist. "grep -R"-ing through the code, it was only mentioned in fold.c. I tried to fix this originally by trying to implement xgetdelim, but the problem is probably that some un-commited changes were made to lib/xwrap.c. which fold.c based itself on. then when _only_ fold.c got committed were errors apparent. I can't get a good fix for this command that looks reasonable, compiles, and passes it's tests; Best I have done was make something that passed the first test case. Rob, If you have some changes you made to xwrap.c. Can you commit them? "make defconfig toybox" no longer works. And everything about this error makes me think this was a "Develop and test fold.c under a uncommited version of xwrap.c with xgetdelim(), then only update fold.c" issue. Especially since versions of fold before it's promotion don't use xgetdelim and compile ceanly - Oliver Webb ___ Toybox mailing list Toybox@lists.landley.net http://lists.landley.net/listinfo.cgi/toybox-landley.net