Bug 1173310 - Source also files below /usr/etc/profile.d
Summary: Source also files below /usr/etc/profile.d
Status: RESOLVED FIXED
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: Basesystem (show other bugs)
Version: Current
Hardware: Other Other
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Dr. Werner Fink
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 1165830
  Show dependency treegraph
 
Reported: 2020-06-24 11:44 UTC by Stefan Dirsch
Modified: 2023-04-06 09:49 UTC (History)
3 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Dirsch 2020-06-24 11:44:30 UTC
I'm currently working on the config file switch /etc --> /usr/etc (find Details on https://en.opensuse.org/openSUSE:Packaging_UsrEtc) and now I'm wondering what should be done with files below /etc/profile.d/* (no changes in aaa_base, bash yet)

Thorsten Kuku wrote:
Yes, this should be moved to /usr/etc/profile.d at best, as the files there are not really configuration files or files an admin should
change.
Comment 1 Stefan Dirsch 2020-06-24 11:47:21 UTC
Adding bugowners of aaa_base and bash to Cc.
Comment 2 Ludwig Nussel 2020-06-24 12:09:51 UTC
Sure, was on my todo in the back of my mind anyways but if you can work on that even better :-)

Since /etc/etc is meant to contain "distribution provided configuration files" and most if not all profile *scripts* are not actually config files at I'd probably move them somewhere below eg /usr/share though. Settings read by profile scripts such as defaults for stuff in /etc/sysconfig may be moved to /usr/etc though.
Anyways, anything will be better than having that stuff in /etc. Go ahead :-)
Comment 3 Stefan Dirsch 2020-06-24 13:55:43 UTC
Seem you misuderstood me. I don't plan on implementing this. ;-)
Comment 4 Dr. Werner Fink 2020-06-26 09:48:43 UTC
With the

commit f11837b30cc9ff8d8da75724480ebe172be77991
commit 3f656fda7f0ae31ba5ad1a0b5c966d52823915a4

in aaa_base repository at least the main rc files of bash and tcsh now search for files below /usr/etc and /etc and those of /etc wins if exists.  Nevertheless I giess that there will be more then those changes as it might be that some of the files below /(usr/)etc had be changes as well
Comment 5 Stefan Dirsch 2020-06-26 11:28:06 UTC
Thanks a lot! Appears to be https://github.com/openSUSE/aaa_base
Comment 6 Stefan Dirsch 2020-08-24 23:54:45 UTC
(In reply to Dr. Werner Fink from comment #4)
> With the
> 
> commit f11837b30cc9ff8d8da75724480ebe172be77991
> commit 3f656fda7f0ae31ba5ad1a0b5c966d52823915a4
> 
> in aaa_base repository at least the main rc files of bash and tcsh now
> search for files below /usr/etc and /etc and those of /etc wins if exists. 
> Nevertheless I giess that there will be more then those changes as it might
> be that some of the files below /(usr/)etc had be changes as well

Looks like this is in aaa_base (factory/TW) since

-------------------------------------------------------------------
Wed Jul 08 10:00:41 UTC 2020 - ro@suse.de

- Update to version 84.87+git20200708.f5e90d7:
  * check for Packages.db and use this instead of Packages
    (boo#1171762)
  * Add also support for /usr/etc/profile.d for tcsh
  * Do add some support for /usr/etc/profile.d
Comment 7 Stefan Dirsch 2020-09-10 10:41:00 UTC
(In reply to Dr. Werner Fink from comment #4)
> in aaa_base repository at least the main rc files of bash and tcsh now
> search for files below /usr/etc and /etc and those of /etc wins if exists. 

Thanks!

> Nevertheless I giess that there will be more then those changes as it might
> be that some of the files below /(usr/)etc had be changes as well

You mean files in /etc/profile.d, which are coming with aaa_base, should be moved to /usr/etc/profile.d as well? I agree with that.
And this bug can be closed.

Files in /etc/profiles.d shipped by other packages should be moved in the appropriate packages. This move is no longer blocked
by aaa_base.
Comment 8 Dr. Werner Fink 2020-09-15 10:10:45 UTC
OK .. fixed for now