#7 Move user bin directories in front of the PATH
Merged a year ago by svashisht. Opened a year ago by churchyard.
rpms/ churchyard/bash user_path_priority  into  master

file modified
+6 -1

@@ -7,7 +7,7 @@ 

  Version: %{baseversion}%{patchleveltag}

  Name: bash

  Summary: The GNU Bourne Again shell

- Release: 1%{?dist}

+ Release: 2%{?dist}

  License: GPLv3+

  Url: https://www.gnu.org/software/bash

  Source0: https://ftp.gnu.org/gnu/bash/bash-%{baseversion}.tar.gz

@@ -316,6 +316,11 @@ 

  %{_libdir}/pkgconfig/%{name}.pc

  

  %changelog

+ * Mon Jun 25 2018 Miro Hrončok <mhroncok@redhat.com> - 4.4.23-2

+ - Move user bin directories in front of the PATH

+   See: https://fedoraproject.org/wiki/Changes/UserPathPrioritization

+   Resolves: #1595098

+ 

  * Tue Jun 12 2018 Siteshwar Vashisht <svashisht@redhat.com> - 4.4.23-1

  - Update to bash-4.4 patchlevel 23

    Resolves: #1585510

file modified
+1 -1

@@ -7,6 +7,6 @@ 

  

  # User specific environment and startup programs

  

- PATH=$PATH:$HOME/.local/bin:$HOME/bin

+ PATH=$HOME/.local/bin:$HOME/bin:$PATH

There can be another flame war on whether $HOME/.local/bin should come first or $HOME/bin. :)

  

  export PATH

This BZ is closed, but by a workaround. The number can be changed to the tracker bug assigned by the wrangler.

rebased onto 329df36

a year ago

There can be another flame war on whether $HOME/.local/bin should come first or $HOME/bin. :)

Haha! I was thinking about the same. My impression is that use of "bin" was kinda deprecated by introduction of .local, in the same way .config folder made use of config files in root of user user HOME deprecated.

Still, as long we manage to ship this we can call it a success, regardless which of two comes first. Still, it will be a long road to get this into CentOS and RHEL on a fast track, if even possible.

Basically this change LGTM. So I will merge it.

Pull-Request has been merged by svashisht

a year ago

long road to get this into CentOS and RHEL

I wouldn't get my hopes up for current ELs to include this change.