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

Fix typos and phrasing in the Web servers howto #52465

Closed
lambertdw mannequin opened this issue Mar 24, 2010 · 27 comments
Closed

Fix typos and phrasing in the Web servers howto #52465

lambertdw mannequin opened this issue Mar 24, 2010 · 27 comments
Assignees
Labels
docs Documentation in the Doc dir easy

Comments

@lambertdw
Copy link
Mannequin

lambertdw mannequin commented Mar 24, 2010

BPO 8218
Nosy @loewis, @birkenfeld, @ezio-melotti, @merwok, @bitdancer, @briancurtin
Files
  • issue8218.diff: Patch against trunk.
  • issue8218_v3.diff: trunk
  • issue8218_v4.diff: Patch against trunk (superset of issue8218_v3.diff)
  • webservers_howto.patch
  • 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/ezio-melotti'
    closed_at = <Date 2010-04-05.04:07:22.966>
    created_at = <Date 2010-03-24.01:43:10.277>
    labels = ['easy', 'docs']
    title = 'Fix typos and phrasing in the Web servers howto'
    updated_at = <Date 2010-04-05.04:07:22.964>
    user = 'https://bugs.python.org/LambertDW'

    bugs.python.org fields:

    activity = <Date 2010-04-05.04:07:22.964>
    actor = 'ezio.melotti'
    assignee = 'ezio.melotti'
    closed = True
    closed_date = <Date 2010-04-05.04:07:22.966>
    closer = 'ezio.melotti'
    components = ['Documentation']
    creation = <Date 2010-03-24.01:43:10.277>
    creator = 'LambertDW'
    dependencies = []
    files = ['16686', '16693', '16694', '16750']
    hgrepos = []
    issue_num = 8218
    keywords = ['patch', 'easy', 'needs review']
    message_count = 27.0
    messages = ['101614', '101615', '101616', '101617', '101618', '101619', '101620', '101621', '101628', '101639', '101781', '101884', '101895', '101896', '101897', '101903', '101912', '101913', '101914', '101917', '102052', '102053', '102054', '102055', '102056', '102287', '102362']
    nosy_count = 9.0
    nosy_names = ['loewis', 'georg.brandl', 'LambertDW', 'ezio.melotti', 'eric.araujo', 'r.david.murray', 'brian.curtin', 'ubershmekel', 'aditya.bhargava']
    pr_nums = []
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue8218'
    versions = ['Python 2.6', 'Python 3.1', 'Python 2.7', 'Python 3.2']

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    http://docs.python.org/py3k/howto/webservers.html

    The path to the interpreter in the shebang (#!/usr/bin/env python) must be currect.

    Sorry man there are a great many distractions here.

    @lambertdw lambertdw mannequin assigned birkenfeld Mar 24, 2010
    @lambertdw lambertdw mannequin added the docs Documentation in the Doc dir label Mar 24, 2010
    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    In same file, "intermangled with Python” and it has a “Publisher” which destignates"

    intermingled?
    designates?

    Thanks, Dave.

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    "Another problem ist the basic concept"

                 is
    

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    much easier as is is specially designed

    more easily and is especially designed

    @ezio-melotti
    Copy link
    Member

    Some more typos:
    "The benefit from this approach is that WSGI applications can be deployed much easier as is is specially designed to host WSGI applications"
    s/is is/it is/

    "Pylons is much like TurboGears with ab even"
    s/ab/an/

    Once you finish reading it I'll fix them all at once.

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    There is a number of middlewares

      are
    

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    you can rely an already existing

       rely on an
    

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    There is an incredible number of frameworks
    s/is/are/

    The majority of users is best
    s/is/are/

    o work together as good as possible.
    s/good/well/

    jects can be saven in a st
    s/saven/saved/

    Thank you. I'm through for now.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Mar 24, 2010

    David, can you provide these changes as a unified diff against the subversion trunk?

    @lambertdw
    Copy link
    Mannequin Author

    lambertdw mannequin commented Mar 24, 2010

    I apologize for the noise.
    Yes, I'll read the article carefully and provide changes as a unified
    diff against the subversion trunk.

    On Wed, 2010-03-24 at 08:20 +0000, Martin v. Löwis wrote:

    Martin v. Löwis <martin@v.loewis.de> added the comment:

    David, can you provide these changes as a unified diff against the subversion trunk?

    ----------
    nosy: +loewis


    Python tracker <report@bugs.python.org>
    <http://bugs.python.org/issue8218\>


    @ezio-melotti
    Copy link
    Member

    See also bpo-8234 that has a patch with some rephrasing too.

    @ezio-melotti
    Copy link
    Member

    Here is a patch that fixes the typos found by David and Yuv and rephrases a couple of parts.

    @merwok
    Copy link
    Member

    merwok commented Mar 29, 2010

    s/framewors/frameworks/g

    @merwok
    Copy link
    Member

    merwok commented Mar 29, 2010

    The rest of the patch seems good to me. Note that “the majority of users is” is as correct as “the majority of users are”, choosing one is a matter of preference.

    @merwok merwok changed the title typo currect Fix typos and phrasing in the Web servers howto Mar 29, 2010
    @bitdancer
    Copy link
    Member

    'is' may be technically correct, however it will sound wrong to a native speaker's ear, at least in America.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Mar 29, 2010

    It seems that a lot of non-native speakers (of English) are participating in this thread. By nature, this is something that native speakers should resolve. Unless I'm mistaken (and Ezio actually *does* speak English as a first language), I propose that one of the native speakers provide a patch (perhaps based on Ezio's patch), and that all non-native speakers abstain from commenting on what they believe is English grammar.

    @briancurtin
    Copy link
    Member

    I went through and made a lot of grammar corrections and rewording throughout the file. See attached patch.

    I have no idea if this doc is technically correct. I only made grammar changes since I'm not a big web developer, but it could probably use a look from someone who is pretty knowledgeable on the topic.

    @merwok
    Copy link
    Member

    merwok commented Mar 30, 2010

    I noticed that “a user” is “he” in the current text; gender-neutral
    language may be more in conformity with the recent diversity statement.

    Regards

    @briancurtin
    Copy link
    Member

    My first patch removed a lot of the he/him/his wording, but now that you mentioned it found another one. Updated the patch.

    @adityabhargava
    Copy link
    Mannequin

    adityabhargava mannequin commented Mar 30, 2010

    Another patch. This contains all the patches from Brian Curtin's issue8218_v3.diff patch along with more edits, including
    an -> in
    teh -> the
    wordspushedtogether spaced out
    unnecessary _a_ words _is_ taken out

    @ezio-melotti
    Copy link
    Member

    There's also an invalid link to an image just after the title "The low-level view":
    -.. .. image:: http.png
    +.. image:: http.png

    The image is not there though, and in r66294 only the document has been added.

    @merwok
    Copy link
    Member

    merwok commented Apr 1, 2010

    This is a reST comment.

    @ezio-melotti
    Copy link
    Member

    If the image is missing, then the whole line can be removed.

    @merwok
    Copy link
    Member

    merwok commented Apr 1, 2010

    Is there a main author for this fileN? If so, they can say whether they left the link as a reminder to put the image back later, or if they were just afraid of real version control <0.4 wink>

    @ezio-melotti
    Copy link
    Member

    The author is Marek Kubica, but he doesn't seem to be registered to this tracker.

    @bitdancer
    Copy link
    Member

    I do know something about web development, so I did a more extensive edit. I'm not saying I've brought it completely up to date or fixed any inaccuracies, but I did change the tone here and there, tightened up a bunch of the language, clarified a few things, and updated a few statements.

    @ezio-melotti
    Copy link
    Member

    Fixed in r79781 (trunk), r79782 (release26-maint), r79783 (py3k) and r79784 (release31-maint).
    Thanks to all!

    @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 easy
    Projects
    None yet
    Development

    No branches or pull requests

    5 participants