From: SourceForge.net <no...@so...> - 2010-03-17 08:50:27
|
Bugs item #2971827, was opened at 2010-03-17 09:50 Message generated for change (Tracker Item Submitted) made by chaoflow You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Florian Friesdorf (chaoflow) Assigned to: Nobody/Anonymous (nobody) Summary: docutils sometimes contains roman Initial Comment: Docutils includes roman, depending on the install-time environment. This raises problems with setups that use egg-caches (e.g. zc.buildout). I think an egg should always provide the same content, independent of the install-time environment. roman is available as an egg via pypi (currently 1.4.0) and differs to the one in docutils only in: --- docutils-0.6-py2.6.egg/roman.py 2010-03-17 09:30:33.029213944 +0100 +++ roman.py 2009-07-23 18:34:18.000000000 +0200 @@ -41,7 +41,7 @@ """convert integer to Roman numeral""" if not (0 < n < 5000): raise OutOfRangeError, "number out of range (must be 1..4999)" - if int(n) != n: + if int(n) <> n: raise NotIntegerError, "decimals can not be converted" result = "" I think docutils should never include roman but require it as a dependency. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 |
From: SourceForge.net <no...@so...> - 2010-03-24 21:46:56
|
Bugs item #2971827, was opened at 2010-03-17 08:50 Message generated for change (Settings changed) made by milde You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Pending >Resolution: Postponed >Priority: 3 Private: No Submitted By: Florian Friesdorf (chaoflow) Assigned to: Nobody/Anonymous (nobody) Summary: docutils sometimes contains roman Initial Comment: Docutils includes roman, depending on the install-time environment. This raises problems with setups that use egg-caches (e.g. zc.buildout). I think an egg should always provide the same content, independent of the install-time environment. roman is available as an egg via pypi (currently 1.4.0) and differs to the one in docutils only in: --- docutils-0.6-py2.6.egg/roman.py 2010-03-17 09:30:33.029213944 +0100 +++ roman.py 2009-07-23 18:34:18.000000000 +0200 @@ -41,7 +41,7 @@ """convert integer to Roman numeral""" if not (0 < n < 5000): raise OutOfRangeError, "number out of range (must be 1..4999)" - if int(n) != n: + if int(n) <> n: raise NotIntegerError, "decimals can not be converted" result = "" I think docutils should never include roman but require it as a dependency. ---------------------------------------------------------------------- >Comment By: Günter Milde (milde) Date: 2010-03-24 21:46 Message: Docutils' source always includes a copy of roman.py. Handling with `setup.py install` depends on whether 'roman' already is available as a Python module at the target system. Distutils allows specifying a dependency since version 2.5 (http://docs.python.org/release/2.5/whatsnew/pep-314.html) while Docutils supports >= 2.3. Hence we have to postpone this solution. A patch that implements "don't put roman.py into binaries" but keeps the "install if required" could be considered. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 |
From: SourceForge.net <no...@so...> - 2010-04-09 02:20:30
|
Bugs item #2971827, was opened at 2010-03-17 08:50 Message generated for change (Comment added) made by sf-robot You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed Resolution: Postponed Priority: 3 Private: No Submitted By: Florian Friesdorf (chaoflow) Assigned to: Nobody/Anonymous (nobody) Summary: docutils sometimes contains roman Initial Comment: Docutils includes roman, depending on the install-time environment. This raises problems with setups that use egg-caches (e.g. zc.buildout). I think an egg should always provide the same content, independent of the install-time environment. roman is available as an egg via pypi (currently 1.4.0) and differs to the one in docutils only in: --- docutils-0.6-py2.6.egg/roman.py 2010-03-17 09:30:33.029213944 +0100 +++ roman.py 2009-07-23 18:34:18.000000000 +0200 @@ -41,7 +41,7 @@ """convert integer to Roman numeral""" if not (0 < n < 5000): raise OutOfRangeError, "number out of range (must be 1..4999)" - if int(n) != n: + if int(n) <> n: raise NotIntegerError, "decimals can not be converted" result = "" I think docutils should never include roman but require it as a dependency. ---------------------------------------------------------------------- >Comment By: SourceForge Robot (sf-robot) Date: 2010-04-09 02:20 Message: This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: Gnter Milde (milde) Date: 2010-03-24 21:46 Message: Docutils' source always includes a copy of roman.py. Handling with `setup.py install` depends on whether 'roman' already is available as a Python module at the target system. Distutils allows specifying a dependency since version 2.5 (http://docs.python.org/release/2.5/whatsnew/pep-314.html) while Docutils supports >= 2.3. Hence we have to postpone this solution. A patch that implements "don't put roman.py into binaries" but keeps the "install if required" could be considered. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 |
From: SourceForge.net <no...@so...> - 2011-12-20 16:41:55
|
Bugs item #2971827, was opened at 2010-03-17 01:50 Message generated for change (Comment added) made by milde You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Closed >Resolution: Fixed Priority: 3 Private: No Submitted By: Florian Friesdorf (chaoflow) Assigned to: Nobody/Anonymous (nobody) Summary: docutils sometimes contains roman Initial Comment: Docutils includes roman, depending on the install-time environment. This raises problems with setups that use egg-caches (e.g. zc.buildout). I think an egg should always provide the same content, independent of the install-time environment. roman is available as an egg via pypi (currently 1.4.0) and differs to the one in docutils only in: --- docutils-0.6-py2.6.egg/roman.py 2010-03-17 09:30:33.029213944 +0100 +++ roman.py 2009-07-23 18:34:18.000000000 +0200 @@ -41,7 +41,7 @@ """convert integer to Roman numeral""" if not (0 < n < 5000): raise OutOfRangeError, "number out of range (must be 1..4999)" - if int(n) != n: + if int(n) <> n: raise NotIntegerError, "decimals can not be converted" result = "" I think docutils should never include roman but require it as a dependency. ---------------------------------------------------------------------- Comment By: Günter Milde (milde) Date: 2011-12-20 08:41 Message: Fixed; thanks for the bug report. You can download a current snapshot from: http://docutils.svn.sourceforge.net/viewvc/docutils/trunk/docutils/?view=tar ---------------------------------------------------------------------- Comment By: SourceForge Robot (sf-robot) Date: 2010-04-08 19:20 Message: This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker). ---------------------------------------------------------------------- Comment By: Günter Milde (milde) Date: 2010-03-24 14:46 Message: Docutils' source always includes a copy of roman.py. Handling with `setup.py install` depends on whether 'roman' already is available as a Python module at the target system. Distutils allows specifying a dependency since version 2.5 (http://docs.python.org/release/2.5/whatsnew/pep-314.html) while Docutils supports >= 2.3. Hence we have to postpone this solution. A patch that implements "don't put roman.py into binaries" but keeps the "install if required" could be considered. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=422030&aid=2971827&group_id=38414 |
From: Guenter M. <mi...@us...> - 2010-03-23 11:42:24
|
On 2010-03-17, SourceForge.net wrote: > Docutils includes roman, depending on the install-time environment. The source always includes a copy of roman.py, however it is only installed if it is not already available at the target system. I agree that this might pose problems with binary distributions. I suppose the inclusion of the setup.py module dates back to times when this dependency could not be auto-solved by distutils. ... > I think docutils should never include roman but require it as a dependency. Agreed. This would also make the setup.py file simpler. Günter |