Bug 1176431 - Get rid of system config files in /etc: x11-tools
Summary: Get rid of system config files in /etc: x11-tools
Status: RESOLVED FIXED
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: X.Org (show other bugs)
Version: Current
Hardware: Other Other
: P3 - Medium : Normal (vote)
Target Milestone: ---
Assignee: Stefan Dirsch
QA Contact: Gfx Bugs
URL:
Whiteboard:
Keywords:
Depends on: 1173321 1177045
Blocks: 1173001
  Show dependency tree / graph
 
Reported: 2020-09-10 20:45 UTC by Stefan Dirsch
Modified: 2023-04-24 09:40 UTC (History)
2 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Dirsch 2020-09-10 20:45:36 UTC 
# rpm -ql x11-tools | grep /etc
/etc/X11/xim
/etc/X11/xim.d
/etc/X11/xim.d/none
/etc/skel/.i18n
/etc/skel/.xim.template
Comment 1 Stefan Dirsch 2020-09-10 20:50:32 UTC 
(In reply to Stefan Dirsch from comment #0)
> /etc/skel/.i18n
> /etc/skel/.xim.template

Blocked by boo#1173321
Comment 2 Stefan Dirsch 2020-09-10 21:31:32 UTC 
> /etc/X11/xim
> /etc/X11/xim.d
> /etc/X11/xim.d/none

moved to /usr/etc/X11
Comment 3 Fuminobu Takeyama 2020-09-20 08:24:14 UTC 
I'm wondering /usr/lib would be appropriate place to move because these files are scripts rather than config files.
Comment 4 Takashi Iwai 2020-09-21 11:51:55 UTC 
(In reply to Fuminobu Takeyama from comment #3)
> I'm wondering /usr/lib would be appropriate place to move because these
> files are scripts rather than config files.

%libexec makes sense, as those are indeed shell scripts, indeed.
OTOH, if the system needs overriding the setup, where is the appropriate place?
/etc over /usr/etc looks normal, but for /usr/lib/blah?
Comment 5 Fuminobu Takeyama 2020-09-21 13:01:46 UTC 
> OTOH, if the system needs overriding the setup, where is the appropriate
> place?

If those scripts were intended to be overridden, they could be considered as config files rather than scripts, and thus, they should be placed under /usr/etc.
Comment 6 Stefan Dirsch 2020-09-22 01:03:58 UTC 
At least /etc/X11/xim.d are.
Comment 7 Fuminobu Takeyama 2020-09-28 15:02:51 UTC 
- /etc/X11/xim.d  -> /usr/etc/X11/xim.d

Ok, let's keep it for config dir so that sysadmins can write custom IM launch scripts.

- /etc/X11/xim
- /etc/X11/xim.d/xim
- /etc/X11/xim.d/ibus  (owned by ibus)
- /etc/X11/xim.d/fcitx (owned by fcitx)
- /etc/X11/xim.d/gcin  (owned by gcin)

These files are scripts. So /usr/libexec/X11/xim (or somewhere like that) will be appropriate.

If someone wants to override IM launch script under ja environment, for example, she can write /etc/X11/xim.d/00-custom. I think she does not need to override the existing script files like /usr/etc/X11/ibus.

# Of course, xim script need to be updated so that it can merge files in both/etc/X11/xim.d/$LANG and /usr/etc/X11/xim.d/$LANG
Comment 8 Stefan Dirsch 2020-09-28 19:40:34 UTC 
Adjusted xim script accordingly.
Comment 9 Stefan Dirsch 2020-10-19 03:19:58 UTC 
(In reply to Stefan Dirsch from comment #1)
> (In reply to Stefan Dirsch from comment #0)
> > /etc/skel/.i18n
> > /etc/skel/.xim.template
> 
> Blocked by boo#1173321

No longer blocked. Now files moved to /usr/etc/skel. Closing.