Project

General

Profile

Bug #7996

/etc/skel/.profile overrides /etc/profile

Added by Ovidiu - Dan Bogat over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
OI-Userland
Target version:
Start date:
2017-03-22
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage

Description

The default .profile that gets created for new users hardwires the PATH variable to a specific path, thus overriding the settings in /etc/profile.

New systemusers also get the hardwired sbin directories into their PATH.


Files

etc_profile.patch (304 Bytes) etc_profile.patch /etc/profile patch Ovidiu - Dan Bogat, 2017-03-22 11:40 AM
etc_skel_profile.patch (336 Bytes) etc_skel_profile.patch /etc/skel/.profile patch Ovidiu - Dan Bogat, 2017-03-22 11:40 AM

History

#1

Updated by Ovidiu - Dan Bogat over 2 years ago

I've included 2 files to patch /etc/profile and /etc/skel/.profile

/etc/profile will check for EUID = 0 and the LOGNAME = "root" to export sbin directories.

/etc/skel/.profile will include the already set PATH variable in its export.

#2

Updated by Aurélien Larcher over 2 years ago

What is your release version?
This seems to have been addressed already, the lines are commented out by default.

#3

Updated by Aurélien Larcher over 2 years ago

See commits referenced in #2931

#4

Updated by Toomas Soome over 2 years ago

Aurélien Larcher wrote:

See commits referenced in #2931

Also note the place to set system wide PATH in illumos is in /etc/default/login.

#5

Updated by Ovidiu - Dan Bogat over 2 years ago

Aurélien Larcher wrote:

What is your release version?
This seems to have been addressed already, the lines are commented out by default.

Well apparently yes and still no... somehow.

I've grabbed the "Minimal Install DVD" (OpenIndiana Hipster 2016.10 Minimal Install DVD (32/64-bit x86)) and burned it to a disk. Downloaded the iso on march 22nd with the Filename: OI-hipster-minimal-20161030.iso.

It had a (locally computed) shasum @ 97f69cf31b6a59ba6c2e7f8726051eee2a5bab52bec00db20927bb35849e3d98 OI-hipster-minimal-20161030.iso
As it should have.

This Image is still the one available at this moment.

I've used that disk to install Openindiana on 2 different machines.

One of the machines still hast this problem (a laptop):

$ uname -a
SunOS dever 5.11 illumos-2816291 i86pc i386 i86pc Solaris

The other machine had this too, yet I ran a "pkg update" and rebooted it afterwards.

  1. uname -a
    SunOS devops 5.11 illumos-a5f6629 i86pc i386 i86pc

This is the stat on /etc/skel/.profile (which has the timestamps of the update)

  1. stat /etc/skel/.profile
    File: '/etc/skel/.profile'
    Size: 377 Blocks: 2 IO Block: 512 regular file
    Device: 40d0002h/67960834d Inode: 433898 Links: 1
    Access: (0644/-rw-r--r--) Uid: ( 0/ root) Gid: ( 1/ other)
    Access: 2017-03-28 22:39:35.942982595 +0200
    Modify: 2017-03-28 11:48:09.354351975 +0200
    Change: 2017-03-28 11:48:09.354460008 +0200
    Birth: 2017-03-28 11:48:09.342017127 +0200

So I guess the issue itself has been fixed.

Aurélien Larcher wrote:

See commits referenced in #2931

I've seen that initially, but it dealt with the prompt specifically.
Admittely I did not check out the patch itself, which also fixed this issue.

Toomas Soome wrote:

Aurélien Larcher wrote:

See commits referenced in #2931

Also note the place to set system wide PATH in illumos is in /etc/default/login.

Thanks. Seeing things deliberately in skel/.profile I thought illumos went a different way here and I didn't want to create to much nosie and didn't pay it that much attention honestly.

------
Conclusion (I've tested it on my 2nd machine which still had the issue from a fresh install):

Fresh Install from OI-hipster-minimal-20161030.iso + "pkg update" + reboot fixed it for me

(so it got fixed with #2931)

#6

Updated by Aurélien Larcher over 2 years ago

The ISO was generated on Oct. 30. 2016 while the fix was merged Nov 5. 2016 as specified in the merge commit.

#7

Updated by Aurélien Larcher over 2 years ago

  • Status changed from New to Closed

Fixed in #2931

Also available in: Atom PDF