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

distutils doc contains lots of XXX #57925

Closed
florentx mannequin opened this issue Jan 5, 2012 · 9 comments
Closed

distutils doc contains lots of XXX #57925

florentx mannequin opened this issue Jan 5, 2012 · 9 comments
Assignees
Labels
docs Documentation in the Doc dir type-feature A feature request or enhancement

Comments

@florentx
Copy link
Mannequin

florentx mannequin commented Jan 5, 2012

BPO 13716
Nosy @birkenfeld, @tarekziade, @merwok, @florentx

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-02-26.03:21:50.271>
created_at = <Date 2012-01-05.19:02:15.569>
labels = ['type-feature', 'docs']
title = 'distutils doc contains lots of XXX'
updated_at = <Date 2012-02-26.03:21:50.269>
user = 'https://github.com/florentx'

bugs.python.org fields:

activity = <Date 2012-02-26.03:21:50.269>
actor = 'eric.araujo'
assignee = 'eric.araujo'
closed = True
closed_date = <Date 2012-02-26.03:21:50.271>
closer = 'eric.araujo'
components = ['Documentation']
creation = <Date 2012-01-05.19:02:15.569>
creator = 'flox'
dependencies = []
files = []
hgrepos = []
issue_num = 13716
keywords = []
message_count = 9.0
messages = ['150683', '150745', '150793', '150798', '150800', '150801', '152686', '154299', '154307']
nosy_count = 6.0
nosy_names = ['georg.brandl', 'tarek', 'eric.araujo', 'flox', 'docs@python', 'python-dev']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue13716'
versions = ['Python 2.7', 'Python 3.2', 'Python 3.3']

@florentx
Copy link
Mannequin Author

florentx mannequin commented Jan 5, 2012

http://docs.python.org/distutils/apiref.html?highlight=XXX#module-distutils.ccompiler

We find lots of "XXX" and "XXX see also." which give no information.

@florentx florentx mannequin assigned docspython Jan 5, 2012
@florentx florentx mannequin added type-bug An unexpected behavior, bug, or error docs Documentation in the Doc dir labels Jan 5, 2012
@merwok
Copy link
Member

merwok commented Jan 6, 2012

Tarek ruled that time spent improving the distutils docs was time lost and better spent on distutils2. Accordingly, I only fix clear bugs in the doc but don’t improve them in any way. I’m inclined to close this as wontfix, or I can remove the XXX if you’re strongly against them. (For the packaging docs, I will fix them properly.)

@birkenfeld
Copy link
Member

I would make comments out of the XXX, and if a whole section is just that XXX, remove the section as well.

@merwok
Copy link
Member

merwok commented Jan 7, 2012

These are the XXX:

XXX true? does ANSI say anything about this?
XXX defaults to what?
XXX see also.
XXX see also.
XXX see also.
XXX see also.
XXX see also.

The first two are questions that can be answered, and when I research them for the packaging docs I could also backport the changes to the distutils docs; what to do for the other instances is less clear, so just deleting may be better than turning them into comments that will never get solved.

@merwok merwok assigned merwok and unassigned docspython Jan 7, 2012
@birkenfeld
Copy link
Member

Agreed.

@florentx
Copy link
Mannequin Author

florentx mannequin commented Jan 7, 2012

The proposed solution is ok.

It sounds like unfinished documentation when you hit an "XXX".
For the "/dev/" documentation, it's OK, but for the released version, we should avoid it.

@python-dev
Copy link
Mannequin

python-dev mannequin commented Feb 5, 2012

New changeset 3d25869fce0c by Éric Araujo in branch '3.2':
Hide or remove user-visible XXX notes from distutils doc (bpo-13716).
http://hg.python.org/cpython/rev/3d25869fce0c

New changeset 1cb9b8126534 by Éric Araujo in branch 'default':
Merge edits from 3.2 (bpo-13716, bpo-1040439, bpo-2945, bpo-13770, bpo-6715)
http://hg.python.org/cpython/rev/1cb9b8126534

@python-dev
Copy link
Mannequin

python-dev mannequin commented Feb 26, 2012

New changeset e853ea9efc6e by Éric Araujo in branch '2.7':
Hide or remove user-visible XXX notes from distutils doc (bpo-13716).
http://hg.python.org/cpython/rev/e853ea9efc6e

@merwok
Copy link
Member

merwok commented Feb 26, 2012

Done.

@merwok merwok closed this as completed Feb 26, 2012
@merwok merwok added type-feature A feature request or enhancement and removed type-bug An unexpected behavior, bug, or error labels Feb 26, 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 type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants