Bug 10758 - /etc/profile.d/30python.sh and /etc/profile.d/30python.csh not marked as config file
Summary: /etc/profile.d/30python.sh and /etc/profile.d/30python.csh not marked as conf...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 3
Hardware: All Linux
Priority: Normal minor
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: advisory has_procedure mga3-64-ok mga...
Keywords: Junior_job, validated_update
Depends on: 11785
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-11 21:19 CEST by Hartmut Goebel
Modified: 2013-12-18 00:52 CET (History)
5 users (show)

See Also:
Source RPM: python-2.7.3-7.mga3.src.rpm and python3-3.3.0-4.mga3.src.rpm
CVE:
Status comment:


Attachments

Description Hartmut Goebel 2013-07-11 21:19:16 CEST
The files /etc/profile.d/30python.sh and /etc/profile.d/30python.csh are not marked as a config file:

$ rpm -q --configfiles python | grep profil
# no result
$ rpm -q --configfiles python3 | grep profil
# no result

Thus these two files are overwritten when updating the package. This leads to problems, if one changed the file.

Changing the wile is necessary since here PYTHONDONTWRITEBYTECODE is set -- which many users do not like (see bug #3348 and https://wiki.mageia.org/en/Feature:PythonByteCompiling)
Manuel Hiebel 2013-07-12 18:05:19 CEST

CC: (none) => makowski.mageia, oliver.bgr

Comment 1 Hartmut Goebel 2013-07-26 10:23:41 CEST
Please note: This bug is in both python2 and python3 packages.
Comment 2 Philippe Makowski 2013-07-26 10:36:17 CEST
will be solved in Mageia 4
can't be done before

Depends on: (none) => 3348

Comment 3 David Walser 2013-08-13 03:09:18 CEST
Fixing the version assignment then.  I'm not sure they really need to be marked as config files though.  Can't sysadmins just drop another file in /etc/profile.d to override it, rather than messing with a packaged file?

Version: 3 => Cauldron

Comment 4 Hartmut Goebel 2013-08-15 12:19:16 CEST
Thanks.

(In reply to David Walser from comment #3)
> I'm not sure they really need to be
> marked as config files though. 

Well, files in /etc are config-files by definition. See 
https://en.wikipedia.org/wiki/Filesystem_Hierarchy_Standard#Directory_structure
https://de.wikipedia.org/wiki/Filesystem_Hierarchy_Standard#.2Fetc_.E2.80.93_spezifische_Konfigurationsdateien
 
> Can't sysadmins just drop another file in
> /etc/profile.d to override it, rather than messing with a packaged file?

This would work for this files, but not in general. Image a profile.d-file executing some command. The sysadmin could not suppress this by adding another config-file (depending on the case, of course).
Comment 5 Philippe Makowski 2013-09-05 20:20:04 CEST
I will remove them from the package
Comment 6 Philippe Makowski 2013-11-11 13:17:20 CET
I have uploaded a patched/updated package for Mageia 3.


Suggested advisory:
========================

Updated python and python3 packages fix mga#10758:

The files /etc/profile.d/30python.sh and /etc/profile.d/30python.csh are now marked as a config file.
/etc/profile.d/30python.sh and /etc/profile.d/30python.csh are now provided only by python not by python3. It will help the upgrade process to mga4.

Updated packages in core/updates_testing:
========================
tkinter3-3.3.0-4.4.mga3.x86_64
python3-debuginfo-3.3.0-4.4.mga3.x86_64
python3-debuginfo-3.3.0-4.4.mga3.i586
python3-3.3.0-4.4.mga3.x86_64
tkinter3-apps-3.3.0-4.4.mga3.x86_64
lib64python3-devel-3.3.0-4.4.mga3.x86_64
tkinter3-apps-3.3.0-4.4.mga3.i586
lib64python3.3-3.3.0-4.4.mga3.x86_64
python3-3.3.0-4.4.mga3.i586
libpython3-devel-3.3.0-4.4.mga3.i586
python3-docs-3.3.0-4.4.mga3.noarch
libpython3.3-3.3.0-4.4.mga3.i586
tkinter3-3.3.0-4.4.mga3.i586

lib64python-devel-2.7.5-1.3.mga3.x86_64
tkinter-2.7.5-1.3.mga3.x86_64
python-docs-2.7.5-1.3.mga3.noarch
tkinter-apps-2.7.5-1.3.mga3.i586
tkinter-2.7.5-1.3.mga3.i586
python-debuginfo-2.7.5-1.3.mga3.x86_64
python-debuginfo-2.7.5-1.3.mga3.i586
lib64python2.7-2.7.5-1.3.mga3.x86_64
libpython2.7-2.7.5-1.3.mga3.i586
python-2.7.5-1.3.mga3.x86_64
libpython-devel-2.7.5-1.3.mga3.i586
tkinter-apps-2.7.5-1.3.mga3.x86_64
python-2.7.5-1.3.mga3.i586


Source RPMs: 
python3-3.3.0-4.4.mga3.src
python-2.7.5-1.3.mga3.src

Depends on: 3348 => (none)
Assignee: bugsquad => qa-bugs

Comment 7 claire robinson 2013-11-11 14:09:26 CET
Thanks Philippe

Version: Cauldron => 3

Comment 8 Dave Hodgins 2013-11-30 14:15:51 CET
Adding bug 11785 as a blocker for this bug. Have to sort out which bug
python 3 should be pushed for.

CC: (none) => davidwhodgins
Depends on: (none) => 11785

Comment 9 Dave Hodgins 2013-12-05 18:20:53 CET
As the version of python3 in updates testing has been replaced by
python3-3.3.0-4.5.mga3.src.rpm for bug 11782, excluding python3 from
the advisory, so this bug report is now only for python-2.

Advisory 10758.adv committed to svn.

Whiteboard: (none) => advisory

Comment 10 claire robinson 2013-12-12 17:24:53 CET
So rpms for this one are now just

lib64python2.7-2.7.5-1.3.mga3.x86_64
lib64python-devel-2.7.5-1.3.mga3.x86_64
libpython2.7-2.7.5-1.3.mga3.i586
libpython-devel-2.7.5-1.3.mga3.i586
python-2.7.5-1.3.mga3.i586
python-2.7.5-1.3.mga3.x86_64
python-debuginfo-2.7.5-1.3.mga3.i586
python-debuginfo-2.7.5-1.3.mga3.x86_64
python-docs-2.7.5-1.3.mga3.noarch
tkinter-2.7.5-1.3.mga3.i586
tkinter-2.7.5-1.3.mga3.x86_64
tkinter-apps-2.7.5-1.3.mga3.i586
tkinter-apps-2.7.5-1.3.mga3.x86_64

from python-2.7.5-1.3.mga3.src.rpm
Comment 11 claire robinson 2013-12-16 14:04:23 CET
testing complete mga3 64 using idle

Whiteboard: advisory => advisory has_procedure mga3-64-ok

Comment 12 claire robinson 2013-12-16 14:08:46 CET
ALso confirmed that the two files mentioned are now config files and only for python not python3

$ rpm -q --configfiles python | grep profil
/etc/profile.d/30python.csh
/etc/profile.d/30python.sh

$ rpm -q --configfiles python3 | grep profil
Comment 13 claire robinson 2013-12-16 14:15:00 CET
Testing complete mga3 32

Validating.

Could sysadmin please push from 3 core/updates_testing to updates

Thanks!

Keywords: (none) => validated_update
Whiteboard: advisory has_procedure mga3-64-ok => advisory has_procedure mga3-64-ok mga3-32-ok
CC: (none) => sysadmin-bugs

Comment 14 Thomas Backlund 2013-12-18 00:52:28 CET
Update pushed:
http://advisories.mageia.org/MGAA-2013-0131.html

Status: NEW => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.