Home_greyopenFATE - openSUSE feature tracking > #311297
Dashboard | Search | Sign up | Login

Please login or register to be able to edit or vote this feature.

Get rid of SuSEconfig.groff

Feature state

openSUSE Distribution
Done

Description

There is no point to run that code after installation of every package (and it's not run when you use zypper or 3rd party tools anyways).

Possible solutions:

  1. Fix groff to respect locale settings (LC_PAPER); ideal solution.
  2. Add that code to yast2 language (write things during installation and whenever the user changes it later); should be easy to do.
  3. Drop SuSEconfig.groff without replacement; unclear consequences.
  4. Add that code to yast2 language (write things only during installation), based on comment #1.

I would vote for 1, as it's a proper solution. If not doable or not doable with a reasonable effort let's at least do 2.

Relations

Discussion


icons/user_comment.png T. K. wrote: (7 years ago)

Option 1 is for various reason no option.

My proposal is based on 2: YaST2 should write /etc/papersize during installation, and only there. Rewriting it later is a bad idea, since the admin could have choosen to change the default.

icons/user_comment.png M. S. wrote: (7 years ago)

Can you elaborate why is 1 not an option? Regarding your proposal it is now changed everytime a new package is installed (unless turned off in sysconfig), so I don't think it would hurt to update it from YaST, but I am fine with both options.

icons/user_comment.png T. K. wrote: (7 years ago)

Read the code, the file is not changed if the administrator has modified it.

icons/user_comment.png M. S. wrote: (7 years ago)

OK. And why is 1 not an option?

icons/user_comment.png T. K. wrote: (7 years ago)

You don't want to have that code in YaST2, this gives only surprises and is a support nightmare, if YaST2 sometimes changes the config and sometimes not.

icons/user_comment.png M. S. wrote: (7 years ago)

That is clear. I wonder why we cannot go with option 1 (fix groff).

icons/user_comment.png T. K. wrote: (7 years ago)

Sorry, misunderstoood. We cannot fix groff because it is not broken. groff has a config file, and that should be honoured. I for example don't want letter output only because I prefer an english locale.

icons/user_comment.png M. S. wrote: (7 years ago)

I would expect you to set LC_PAPER correctly then. But anyways, I can live with option 4.

icons/user_comment.png M. V. wrote: (7 years ago)

SuSEconfig.groff in sle-11-sp1 version overwrites the /etc/papersize everytime - see
Bug #644467 . The fix is easy (I assume the check_md5_and_move is in SLE-11-SP1). So in case you decided to stay with this script, this bug should be fixed.

icons/user_comment.png T. K. wrote: (6 years ago)

For openSUSE: We can break with everything, so I suggest to remove the SuSEconfig.groff file completly without replacement.

Background: groff does not read this file by default, you have to specify it on the command line.

For SLE11 SP2 this is not an option, since we don't know if there aren't scripts using that option and expecting that the file exists.

icons/user_comment.png M. V. wrote: (6 years ago)

Removed together with 1.21 update - sent to Factory by request 74706. Marked as done for openSUSE and SLE-12.

Last change: 3 years ago
Voting
Score: 0
  • Negative: 0
  • Neutral: 0
  • Positive: 0
Tags
Feature Export
Application-xmlXML   Text-x-logPlaintext   PrinterPrint