Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 629846 - dev-util/plan9port: rename 30plan9 to 60plan9
Summary: dev-util/plan9port: rename 30plan9 to 60plan9
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Anthony Basile
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-04 01:33 UTC by Benda Xu
Modified: 2017-09-04 06:40 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Benda Xu gentoo-dev 2017-09-04 01:33:00 UTC
Hi William,

I find renaming 00basic to 50baselayout introduces some unexpected bugs.  For example, plan9port installs env.d/30plan9 containing PATH=/usr/lib/plan9/bin. That PATH should come after baselayout because `ls`, `tar`, etc. of plan9 is not compatible with GNU coreutils.

I wonder the reasoning behind "50" in 50baselayout. Making it 00baselayout will avoid these potential breakages.

Yours,
Benda
Comment 1 Benda Xu gentoo-dev 2017-09-04 01:41:12 UTC
Ah-ha, because of bug 621334.  So the number of plan9port had better to be tuned. I think an announcement in the mailing list is necessary to allow people to adopt their packages to >=baselayout-2.4.
Comment 2 Benda Xu gentoo-dev 2017-09-04 01:45:52 UTC
Hi Andy and Anthony,

From the baselayout-2.4, the system default paths are coded in /etc/env.d/50baselayout.  Plan9 paths had better append to it because of name collisions against GNU coreutils.  I think renaming 30plan9 to 60plan9 is the best way to move forward.  What do you think?

Yours,
Benda
Comment 3 Mike Gilbert gentoo-dev 2017-09-04 01:51:50 UTC
(In reply to Benda Xu from comment #1)
> Ah-ha, because of bug 621334.  So the number of plan9port had better to be
> tuned. I think an announcement in the mailing list is necessary to allow
> people to adopt their packages to >=baselayout-2.4.

Well, it was actually due to bug 255695, but yes, the change was intentional to allow some packages to place their PATH ahead of baselayout.
Comment 4 Anthony Basile gentoo-dev 2017-09-04 01:54:02 UTC
(In reply to Benda Xu from comment #2)
> Hi Andy and Anthony,
> 
> From the baselayout-2.4, the system default paths are coded in
> /etc/env.d/50baselayout.  Plan9 paths had better append to it because of
> name collisions against GNU coreutils.  I think renaming 30plan9 to 60plan9
> is the best way to move forward.  What do you think?
> 
> Yours,
> Benda

sounds reasonable to me.  i don't even know how we choose 30.
Comment 5 Benda Xu gentoo-dev 2017-09-04 06:40:34 UTC
Committed with a revision bump.  Thanks for all your responds.