Bug 106360 - [PATCH] document ~/.local/bin
Summary: [PATCH] document ~/.local/bin
Status: RESOLVED MOVED
Alias: None
Product: Specifications
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Allison Lortie (desrt)
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-05-02 17:43 UTC by Lennart Poettering
Modified: 2019-02-16 12:29 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
basedir: Reformat text and indentation, fix inconsistent hyphenation (5.07 KB, patch)
2018-05-02 17:43 UTC, Lennart Poettering
Details | Splinter Review
basedir: document ~/.local/bin (2.35 KB, patch)
2018-05-02 17:43 UTC, Lennart Poettering
Details | Splinter Review
basedir-spec: correct Allison's name (875 bytes, patch)
2018-05-02 17:44 UTC, Lennart Poettering
Details | Splinter Review

Description Lennart Poettering 2018-05-02 17:43:19 UTC
Created attachment 139280 [details] [review]
basedir: Reformat text and indentation, fix inconsistent hyphenation

This patch set is based on the two patches originally put together by 
Johannes Löthberg:

https://lists.freedesktop.org/archives/xdg/2017-August/013938.html

I promised to commit these, but never did. Let's get this done now.

I discussed this with Allison Karlitskaya (as one of the original spec authors) and we agreed that it might be wise not to introduce yet another set of env vars, but simply document that ~/.local/bin is suitable for dropping binaries in. Hence I reworked Johannes' patches accordingly, and dropped all mentions of any new XDG env vars for this. Instead I added a reference to $PATH however, to clarify that the search path concept already exists and with a recommendation that distributions include ~/.local/bin in it.

I also toned down the wording a bit, from "shall" to "may", to make this more compatible with other views of the world.

Anyway, I hope this is enough to get in.

Josh Triplett reviewed the original patch and liked it, I hope he's OK with this version too.

https://lists.freedesktop.org/archives/xdg/2017-August/013943.html
Comment 1 Lennart Poettering 2018-05-02 17:43:50 UTC
Created attachment 139281 [details] [review]
basedir: document ~/.local/bin
Comment 2 Lennart Poettering 2018-05-02 17:44:18 UTC
Created attachment 139282 [details] [review]
basedir-spec: correct Allison's name
Comment 3 Lennart Poettering 2018-05-02 17:46:54 UTC
(A note: I am not sure how responsive this bugzilla component really is. given that this was discussed on the XDG mailing list already, and got some positive feedback there, and I got some generally OK feedback from Allison too I will commit this in a few days, if I don't hear from anyone else about this and noone else beats me to it)
Comment 4 David Faure 2018-05-07 15:24:44 UTC
Comment on attachment 139281 [details] [review]
basedir: document ~/.local/bin

There's a typo: achitecture --> architecture (twice).

I'm surprised by the last paragraph which says "we know there's a problem, we're not fixing it, just keep it in mind". Shouldn't it say rather something like "if you really need to support multiple architectures, consider using arch-specific directories and modifying PATH accordingly" ?

Looks good otherwise.
Comment 5 GitLab Migration User 2019-02-16 12:29:18 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xdg/xdg-specs/issues/14.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.