[guest@localhost ~]$ python3
Python 3.3.0 (default, Sep 29 2012, 22:07:38)
[GCC 4.7.2 20120921 (Red Hat 4.7.2-2)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> license()
See http://www.python.org/3.3/license.html
>>>

404


answer from webmaster@python.org:

Hello,

When I use the version of Python distributed by python.org and type "license()" I get the full license text and not the url.

It seems like this might be a change made by Red Hat? Either way, the proper place to discuss issues like this is on the Python bug tracker:

	http://bugs.python.org/

Feel free to report an issue there and the developers can look at it.

This email address is actually for reporting problems with the Python.org website!

All the best,

Michael Foord



in Lib/site.py:
[guest@localhost cpython]$ sed -n '453,456p' Lib/site.py
    builtins.license = _Printer(
        "license", "See http://www.python.org/%.3s/license.html" % sys.version,
        ["LICENSE.txt", "LICENSE"],
        [os.path.join(here, os.pardir), here, os.curdir])
[guest@localhost cpython]$

This broke because we now have release 3.3.0 instead of 3.3.
But it's easy to add a redirect, which I've done now.

In the future, site.py should be fixed to say 3.3.0 or 3.3.X depending on the current bugfix version.

For anyone who wants to work on this: the license URL is printed if only if the license file can't be found.

It would be nice to add a test (guarded by the network resource) that checks that the generated url in license is not a 404.  Would you be interested in tackling that, Berker?

New changeset a723adeb1f47 by Senthil Kumaran in branch '3.3':
Fix License URL display and add test to check for license url presence.
http://hg.python.org/cpython/rev/a723adeb1f47

New changeset 12aaf64feca2 by Senthil Kumaran in branch 'default':
merge from 3.3
http://hg.python.org/cpython/rev/12aaf64feca2

Fixed in the 3.3 and 3.4.

3.3 - changeset a723adeb1f47
3.4 - changeset 12aaf64feca2
3.4 - (Fixed a merge conflict) changeset 6ecdf2b3192b

Senthil Kumaran changed the pattern

from: r'^See (http://www\.python\.org/[^/]+/license.html)$'
  to: r'^See (http://www\.python\.org/download/releases/[^/]+/license/)$'

test doesn't pass

[guest@localhost cpython]$ ./python
Python 3.4.0a2+ (default, Sep 14 2013, 05:56:46) 
[GCC 4.7.2 20121109 (Red Hat 4.7.2-8)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> license()
See http://www.python.org/3.4/license.html
>>>


make test
...
[265/380/3] test_site
test test_site failed -- Traceback (most recent call last):
  File "/home/guest/tmp/tests/misc/git/example/cpython/Lib/test/test_site.py", line 416, in test_license_page
    self.assertIsNotNone(mo, msg='can\'t find appropriate url in license')
AssertionError: unexpectedly None : can't find appropriate url in license
...

The test needs to be fixed so that it runs even when the LICENSE file exists. (It is also missing the skip when the network resource is not asserted.)  I have a patched test, and a fix for the 3.4 problem, but I can't apply it because the 3.4.0 license file does not yet exist on the website.

On the other hand, this test could be a pain before the first alpha is released (the web URL doesn't exist yet), so perhaps the test also needs to special case that somehow.

R. David Murray wrote:

>It is also missing the skip when the network resource is not asserted.

How it connects, I copied from another python tests.
The test was skipped without network connection.

"support.requires('network')" ?

>the 3.4.0 license file does not yet exist on the website

http://www.python.org/download/releases/3.4.0/license/
opens

>The test needs to be fixed so that it runs even when the LICENSE file exists.

The fixture can rename it back and forth.

The patch needs to be compatible with version 2.7
Now there urllib.request is importing only.

New changeset 7c18b799841e by R David Murray in branch '3.3':
#18206: Fix test for existence of license URL.
http://hg.python.org/cpython/rev/7c18b799841e

New changeset 321c3c9cd1b5 by R David Murray in branch 'default':
Merge #18206: Fix test for existence of license URL.
http://hg.python.org/cpython/rev/321c3c9cd1b5

New changeset 9354325d2ee4 by R David Murray in branch 'default':
#18206: Re-fix license URL.
http://hg.python.org/cpython/rev/9354325d2ee4

You are right, there was a further typo in the 3.4 URL that I missed.

Yes, requires_resource('network') is what I was referring to (though using transient_internet is also a very good idea :).  I also changed the urllib call to only do a HEAD request.  No need to rename the LICENSE file, I just pulled the info out of the data structure.  It's a white box test, but then so is parsing the "See" line at all, so I think it is acceptable.

(I wonder if it would be worthwhile to have transient_internet call requires_resource('network')?)

Thanks for making these changes (and correcting some of my mistakes), RDM.
They are much better now.

On Sat, Sep 14, 2013 at 10:38 AM, R. David Murray <report@bugs.python.org>wrote:

>
> R. David Murray added the comment:
>
> You are right, there was a further typo in the 3.4 URL that I missed.
>
> Yes, requires_resource('network') is what I was referring to (though using
> transient_internet is also a very good idea :).  I also changed the urllib
> call to only do a HEAD request.  No need to rename the LICENSE file, I just
> pulled the info out of the data structure.  It's a white box test, but then
> so is parsing the "See" line at all, so I think it is acceptable.
>
> (I wonder if it would be worthwhile to have transient_internet call
> requires_resource('network')?)
>
> ----------
> status: open -> closed
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <http://bugs.python.org/issue18206>
> _______________________________________
>