Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove packaging from the stdlib #59352

Closed
pitrou opened this issue Jun 22, 2012 · 15 comments
Closed

Remove packaging from the stdlib #59352

pitrou opened this issue Jun 22, 2012 · 15 comments
Assignees
Labels
docs Documentation in the Doc dir release-blocker stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error

Comments

@pitrou
Copy link
Member

pitrou commented Jun 22, 2012

BPO 15147
Nosy @birkenfeld, @pitrou, @tarekziade, @merwok

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = 'https://github.com/merwok'
closed_at = <Date 2012-07-22.21:21:38.203>
created_at = <Date 2012-06-22.23:58:09.134>
labels = ['type-bug', 'library', 'release-blocker', 'docs']
title = 'Remove packaging from the stdlib'
updated_at = <Date 2012-07-22.21:21:38.201>
user = 'https://github.com/pitrou'

bugs.python.org fields:

activity = <Date 2012-07-22.21:21:38.201>
actor = 'eric.araujo'
assignee = 'eric.araujo'
closed = True
closed_date = <Date 2012-07-22.21:21:38.203>
closer = 'eric.araujo'
components = ['Documentation', 'Library (Lib)', 'Distutils2']
creation = <Date 2012-06-22.23:58:09.134>
creator = 'pitrou'
dependencies = []
files = []
hgrepos = []
issue_num = 15147
keywords = []
message_count = 15.0
messages = ['163504', '163512', '163574', '163649', '163699', '163700', '163710', '163711', '163712', '163746', '163759', '163763', '163818', '166025', '166172']
nosy_count = 6.0
nosy_names = ['georg.brandl', 'pitrou', 'tarek', 'eric.araujo', 'alexis', 'Ramchandra Apte']
pr_nums = []
priority = 'release blocker'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue15147'
versions = ['Python 3.3']

@pitrou
Copy link
Member Author

pitrou commented Jun 22, 2012

Eric, this is for you. If you do only one thing before the 3.3 beta, please do this.

@pitrou pitrou added stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error labels Jun 22, 2012
@merwok
Copy link
Member

merwok commented Jun 23, 2012

Yep, that was on my planning for tonight or tomorrow afternoon.

@merwok merwok added the docs Documentation in the Doc dir label Jun 23, 2012
@birkenfeld
Copy link
Member

Very good, thanks.

@merwok
Copy link
Member

merwok commented Jun 23, 2012

Just a note: a this time I am cleaning the apartment that I’m leaving, so when I say “Saturday afternoon” it’s in Montreal time, so you may see it as early Sunday from Europe. So don’t worry if your Saturday night comes and I did not commit.

@merwok
Copy link
Member

merwok commented Jun 24, 2012

On it.

@merwok
Copy link
Member

merwok commented Jun 24, 2012

The sysconfig module was modified (and sysconfig.cfg was introduced) to serve the needs of the resource system (install_data putting files into system-configured locations and packaging.database finding them); one of the thousand things mentioned in the “Status of Packaging” thread was an opinion by PJE that this was a bad choice, and even if I disagree (being a Debian guy and all :), a PEP would be in order to decide on such a big new functionality. Should these changes be reverted too? (I’d think yes.)

@merwok
Copy link
Member

merwok commented Jun 24, 2012

Packaging removed in 576b8b182039. I’m going to wait an hour or two and in the absence of feedback will remove sysconfig.cfg too; that will be easy to revert if consensus was to keep it and I was not online (but I really think sysconfig.cfg should go).

I’ll also look at the buildbots.

@RamchandraApte
Copy link
Mannequin

RamchandraApte mannequin commented Jun 24, 2012

Why are you removing packaging?

@RamchandraApte
Copy link
Mannequin

RamchandraApte mannequin commented Jun 24, 2012

Oh.. understand now packaging is being deferred.

@merwok
Copy link
Member

merwok commented Jun 24, 2012

sysonfig.cfg removed in e989e7586da6.

@merwok merwok closed this as completed Jun 24, 2012
@pitrou
Copy link
Member Author

pitrou commented Jun 24, 2012

The sysconfig module was modified (and sysconfig.cfg was introduced)
to serve the needs of the resource system (install_data putting files
into system-configured locations and packaging.database finding them);
one of the thousand things mentioned in the “Status of Packaging”
thread was an opinion by PJE that this was a bad choice, and even if I
disagree (being a Debian guy and all :), a PEP would be in order to
decide on such a big new functionality. Should these changes be
reverted too? (I’d think yes.)

What is sysconfig.cfg?

Keeping sysconfig is ok IMO.

@tarekziade
Copy link
Mannequin

tarekziade mannequin commented Jun 24, 2012

We should keep sysconfig imo

@tarekziade tarekziade mannequin reopened this Jun 24, 2012
@merwok
Copy link
Member

merwok commented Jun 24, 2012

What is sysconfig.cfg?
A file containing the install schemes for packaging (not for distutils, these are still in distutils.commands.install) and the categories for data files, to allow installation to various directories for scripts, doc, etc (cf. FHS). The idea AFAICT was to allow OS-level customization of the install paths with a config file instead of code patches. There are already complaints about the sysconfig module API, so I wanted to avoid locking us into this new sysconfig.cfg file format without a longer discussion:

  • is this file needed for built-from-source Pythons installed in e.g. /usr/local?
  • should it apply to distributions installed using non-OS packaging tools with the OS Python?
  • does it remove the need for the --layout=deb gross patch that you can see for example in Debian and derivatives? (i.e. a layout used to build Python projects in a debian/tmp/usr subdirectory of the source package, before using non-Python packaging tools to move these files to the right location)

These are the kind of questions from the top of my head which I think haven’t been hashed out. Or maybe it’s perfectly clear between Tarek and the few OS packagers that discussed this matter but there is no documentation of that. As the only client of the sysconfig.cfg file was packaging, I think the safe thing to do is to not add it in 3.3 and make it a part of the discussions for 3.4 (i.e. may need a PEP).

@birkenfeld
Copy link
Member

Eric, ping? When can this be closed?

@merwok
Copy link
Member

merwok commented Jul 22, 2012

packaging (and related changes in sysconfig) have been removed.

@merwok merwok closed this as completed Jul 22, 2012
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir release-blocker stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

3 participants