Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 489644

Summary: app-shells/fish-2.1.0 version bump
Product: Gentoo Linux Reporter: Elis Axelsson <elis.axelsson>
Component: New packagesAssignee: Jeff (JD) Horelick (RETIRED) <jdhore>
Status: RESOLVED FIXED    
Severity: enhancement CC: polynomial-c
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 488518    
Attachments: Ebuild for fish 2.1.0
Patch that fixes the docdir bug
diff -u of my new ebuild for fish

Description Elis Axelsson 2013-10-28 09:20:00 UTC
fish-2.1.0 was just released, please bump. I made minor adjustments to the ebuild to make it work as expected.

Reproducible: Always
Comment 1 Elis Axelsson 2013-10-28 09:25:21 UTC
Created attachment 362116 [details]
Ebuild for fish 2.1.0

I been using it on the testing version of fish 2.1.0 and it seems to be fine.

Changes I made was to remove the part where we specified docdir to econf to be /usr/share/doc/fish-2.1.0/ since several fish-scripts/functions expects the docs to be in /usr/share/doc/fish/ (for example the function "help"). This should be considered as a bug in fish. But the same bug was there in the 2.0.0 release.

I also removed the declaration of $S, since it seems to not be needed for this release.
Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2013-10-28 09:53:04 UTC
Please attach unified diffs (diff -u) when you did changes to an ebuild. That would be much more handy for us.

Gentoo policy says that a package must install its docs into /usr/share/doc/packagename-fullversion/
So removing this from the ebuild is the wrong approach. Rather fix the scripts which don't respect the docdir variable.
Comment 3 Elis Axelsson 2013-10-28 11:05:38 UTC
Linked this issue on fish-irc, and the devs noticed the docdir part and reported it themselfs: https://github.com/fish-shell/fish-shell/issues/1065

It won't be fixed before 2.2.0, but I could backport it when they patch it. I will also submit a diff -u when I got a patch for this issue.
Comment 4 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2013-10-28 11:42:09 UTC
(In reply to Elis Axelsson from comment #3)
> Linked this issue on fish-irc, and the devs noticed the docdir part and
> reported it themselfs: https://github.com/fish-shell/fish-shell/issues/1065
> 
> It won't be fixed before 2.2.0, but I could backport it when they patch it.
> I will also submit a diff -u when I got a patch for this issue.

Thank you for reporting this issue upstream. :)
Comment 5 Elis Axelsson 2013-10-28 17:13:51 UTC
Created attachment 362154 [details, diff]
Patch that fixes the docdir bug

These changes was commited upstream earlier today, they fix the docdir bug.
Comment 6 Elis Axelsson 2013-10-28 17:14:51 UTC
Created attachment 362156 [details, diff]
diff -u of my new ebuild for fish

Added the usage of the patch submited in previous comment.