View Issue Details

IDProjectCategoryView StatusLast Update
0005122GNUnetotherpublic2018-06-07 00:24
Reporternikita Assigned Tonikita  
PrioritylowSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Summary0005122: gnurl: make the FIXMEs which are required for certain systems obsolete
Description
        # XXX Fix this upstream: Rename include/curl to include/gnurl
        mv include/curl include/gnurl
        # FIX: Tune explicit paths in source files (~400 lines)
        grep -ERl 'include(.*curl/|/curl)' | \
            xargs sed -i -r \
                  -e 's:include/curl:include/gnurl:g' \
                  -e 's:(include.*)curl/:\1gnurl/:g' || die
        # FIX: Tune relative 'curl' path in makefiles under include/
        grep -Rl 'SUBDIRS\s*=\s*curl' | \
            xargs sed -i -r 's:(SUBDIRS\s*=\s*)curl:\1@PACKAGE@:g' || die
        # FIX: Tune relative 'curl' path in makefiles, install phase
        grep -Rl 'pkgincludedir\s*=\s*.*curl' | \
            xargs sed -i -r 's:(pkgincludedir\s*=\s*.*)curl:\1@PACKAGE@:g' || die
        # FIX: Skip examples and man3
        grep -Rl 'SUBDIRS\s*=\s*.*libcurl' docs/ | \
            xargs sed -i -r '/SUBDIRS\s*=\s*.*libcurl/s:^:#:g' || die
Additional InformationJust a reminder so that I can fix it with the next possible release/releases.
TagsNo tags attached.

Activities

nikita

2017-08-19 14:04

reporter   ~0012379

https://bugs.gentoo.org/show_bug.cgi?id=518632 writes:

 Vince C. 2014-07-31 01:09:46 UTC

Created attachment 381922 [details]
gnurl 7.35.0 (stable, latest)

GNUnet recommends gnurl as a dependency, which is (conjectures apart) a rough fork of curl. This ebuild has an optional USE flag to enable DANE certificate validation with GNUTLS. Without gnurl the hostlist daemon will not be included in GNUnet although it appears to be a crucial component.

Note that this ebuild is equivalent to a parallel installation of curl so there are conflicts. Those have been resolved by tuning files in the source tree directly. This is a *very* ugly hack and IMHO the fork maintainer should apply the required changes, such as renaming *curl* into *gnurl* whenever appropriate.

Therefore man3 pages have been skipped — leaving only man1 — as well as the aclocal.m4 file and example files. The include/curl directory is renamed into include/gnurl and dependencies updated accordingly, both in the source tree and in the image install directory.

Although the ebuild is a thoroughly updated version of curl.ebuild, the KEYWORDS section has been limited to x86 architecture as there's no indication yet it compiles and runs properly on other architectures.

This ebuild has been successfully compiled on a Gentoo AMD64 architecture.



---------
Note/Reminder:

It would be good if people would actually communicate with upstreams and not just keep the discussion inside their projects.
If you happen to find this bug, and have further complains about gnURL, report them.

nikita

2017-08-20 16:47

reporter   ~0012380

I can fix this to some degree.
I don't have all the hardware to test if my changes will work on every platform. All I ever do is test on x86_64 and watch hydra.gnu.org for every other supported platform of Guix, which is my primary test platform (because I like to spend less time messing around with non-reproducible errors).

In case the soon to be releases 7.55.1 of gnURL crashes on some platforms: submit your non-system specific fixes!

nikita

2017-08-22 20:53

reporter   ~0012388

Should be solved now with the release I'm about to make. If problems remain, open a new bug.

Issue History

Date Modified Username Field Change
2017-08-19 14:00 nikita New Issue
2017-08-19 14:00 nikita Status new => assigned
2017-08-19 14:00 nikita Assigned To => nikita
2017-08-19 14:04 nikita Note Added: 0012379
2017-08-20 16:47 nikita Note Added: 0012380
2017-08-22 20:53 nikita Note Added: 0012388
2017-08-22 20:53 nikita Status assigned => resolved
2017-08-22 20:53 nikita Resolution open => fixed
2018-06-07 00:24 Christian Grothoff Status resolved => closed