GNU nano - Bugs: bug #50447, nano 2.7.5 fails to build with...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50447: nano 2.7.5 fails to build with musl due to missing support for REG_STARTEND
Submitter: | anon ymous <altf2> | ||
Submitted: | Fri 03 Mar 2017 06:01:55 PM UTC | ||
Severity: | 3 - Normal | Status: | Fixed |
Assigned to: | bens | Open/Closed: | Closed |
Jump to the original submission
Sun 02 Apr 2017 09:50:32 AM UTC, comment #17: |
Benno Schulenberg <bens> |
Thu 16 Mar 2017 06:39:36 PM UTC, comment #16: Thanks for confirming that you can now compile nano, altf2. Could you also verify that searching for a regex (for example for \b) works correctly?
|
Benno Schulenberg <bens> |
Sun 12 Mar 2017 09:31:04 PM UTC, comment #15: it isn't just for REG_STARTEND. if you look at the series of commits landed, it allowed us to drop a lot of custom/buggy shim layers in nano, and become easier to port to more platforms (like Windows). it also keeps us from adding even more (which people were proposing), and it let's us continue to use newer features that are available in the GNU world but not elsewhere. i think you severely underestimate the burden of duplicating gnulib, or of disabling functionality when a platform is missing features.
|
Mike Frysinger <vapier> |
Sun 12 Mar 2017 03:38:00 PM UTC, comment #14: ok, i can confirm it works now (missing xsize was due to a bug in the join impl i used). however i think having to use gnulib just for the sake of REG_STARTEND is a high price to pay:
|
anon ymous <altf2> |
Wed 08 Mar 2017 08:51:21 PM UTC, comment #13: considering your first run failed due to tool compatibility issues, try deleting the lib/ dir entirely and then run the gnulib-tool. if xsize still isn't pulled in to lib/, then you can try debugging that tool and figure out why.
|
Mike Frysinger <vapier> |
Wed 08 Mar 2017 07:20:56 PM UTC, comment #12: there's nothing in the autogen.sh output talking about xsize, and it's not mentioned in the module list in autogen.sh.
|
anon ymous <altf2> |
Wed 08 Mar 2017 07:17:23 PM UTC, comment #11: xsize is a gnulib module. it should be created as part of the ./autogen.sh step. if it wasn't, go back and look at the output of gnulib and see where it's going wrong. |
Mike Frysinger <vapier> |
Wed 08 Mar 2017 07:07:59 PM UTC, comment #10: xsize.h is in the gnulib/ dir (git checkout), but not in the lib/ dir make enters. |
anon ymous <altf2> |
Wed 08 Mar 2017 07:00:15 PM UTC, comment #9: xsize is a gnulib module. it should be created as part of the ./autogen.sh step. if it wasn't, go back and look at the output of gnulib and see where it's going wrong. |
Mike Frysinger <vapier> |
Wed 08 Mar 2017 06:22:13 PM UTC, comment #8: thanks. that alone didn't work, i also had to change doc/Makefile.
|
anon ymous <altf2> |
Wed 08 Mar 2017 03:24:36 PM UTC, comment #7: Yes, when building from git, you need texinfo installed -- see README.GIT. If you don't want to bother with that, just touch doc/nano.html and carry on. |
Benno Schulenberg <bens> |
Wed 08 Mar 2017 03:02:48 PM UTC, comment #6: i installed another join tool and got it to run through.
|
anon ymous <altf2> |
Wed 08 Mar 2017 07:08:11 AM UTC, comment #5: please report your troubles to the upstream gnulib list instead. nano isn't the support point for issues with it. |
Mike Frysinger <vapier> |
Wed 08 Mar 2017 05:42:48 AM UTC, comment #4: i tried to build from git, however when i run autogen.sh gnulib-tool goes into an infinite loop:
|
anon ymous <altf2> |
Tue 07 Mar 2017 11:35:47 PM UTC, comment #3: if you have porting problems with gnulib, then report them to gnulib and get them fixed. a large number of projects (both in GNU and beyond) rely on gnulib as its porting layer. expecting every project out there to implement their own OS shims/compat layers is unreasonable. the end result is going to be even worse than gnulib in terms of bugs and waste of developer time.
|
Mike Frysinger <vapier> |
Tue 07 Mar 2017 06:09:21 AM UTC, comment #2: glad to hear you're willing to fix it, however i doubt that gnulib is a good solution: https://gitlab.com/sortix/sortix/wikis/Gnulib
|
anon ymous <altf2> |
Sun 05 Mar 2017 04:55:10 PM UTC, comment #1: Thanks for reporting. I was curious whether there would be any systems that don't support REG_STARTEND. :)
|
Benno Schulenberg <bens> |
Fri 03 Mar 2017 06:01:55 PM UTC, original submission:
utils.c: In function 'strstrwrapper':
|
anon ymous <altf2> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 10 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-04-02 | bens | Open/Closed | Open | Closed | |
Summary | nano 2.7.5 build fails w/musl due to missing support for REG_STARTEND | nano 2.7.5 fails to build with musl due to missing support for REG_STARTEND | |||
2017-03-16 | bens | Open/Closed | Closed | Open | |
2017-03-12 | vapier | Status | Ready For Test | Fixed | |
Open/Closed | Open | Closed | |||
2017-03-08 | vapier | Severity | 4 - Important | 3 - Normal | |
Status | None | Ready For Test | |||
2017-03-07 | vapier | Summary | regression: build of latest release 2.7.5 fails with musl libc | nano 2.7.5 build fails w/musl due to missing support for REG_STARTEND | |
2017-03-05 | bens | Severity | 3 - Normal | 4 - Important | |
Assigned to | None | bens |
Powered by Savane 3.13-f8d8.
Corresponding source code
The use if gnulib was released in 2.8.0.