[PATCH] Better packaging with setuptools/distutil

7 views
Skip to first unread message

Dimitris Glezos

unread,
Jul 28, 2008, 8:55:29 PM7/28/08
to Dimitris Glezos, Transifex Development List
10 files changed, 307 insertions(+), 57 deletions(-)
MANIFEST.in | 6
ez_setup.py | 233 +++++++++++++++++++++++++++++++
setup.py | 75 +++++++++
transifex.egg-info/PKG-INFO | 18 --
transifex.egg-info/SOURCES.txt | 25 ---
transifex.egg-info/dependency_links.txt | 1
transifex.egg-info/not-zip-safe | 1
transifex.egg-info/paster_plugins.txt | 2
transifex.egg-info/requires.txt | 2
transifex.egg-info/top_level.txt | 1


f-pypi-packaging.patch

Dimitris Glezos

unread,
Jul 28, 2008, 9:41:05 PM7/28/08
to Transifex Development List

Testing building/installing on clean systems needed. Eg.

python setup.py sdist, bdist, bdist_egg, devel, install
rm -rf dist && python setup.py sdist && cd dist && tar zxf
*.tar.gz && cd transifex-0.3 && python setup.py install

-d

--
Dimitris Glezos
Jabber ID: gle...@jabber.org, GPG: 0xA5A04C3B
http://dimitris.glezos.com/

"He who gives up functionality for ease of use
loses both and deserves neither." (Anonymous)
--

Christos Τrochalakis

unread,
Jul 29, 2008, 3:58:50 AM7/29/08
to transif...@googlegroups.com
On Tue, Jul 29, 2008 at 4:41 AM, Dimitris Glezos <dimi...@glezos.com> wrote:
>
> On Tue, Jul 29, 2008 at 3:55 AM, Dimitris Glezos <dimi...@glezos.com> wrote:
>> 10 files changed, 307 insertions(+), 57 deletions(-)
>> MANIFEST.in | 6
>> ez_setup.py | 233 +++++++++++++++++++++++++++++++
>> setup.py | 75 +++++++++
>> transifex.egg-info/PKG-INFO | 18 --
>> transifex.egg-info/SOURCES.txt | 25 ---
>> transifex.egg-info/dependency_links.txt | 1
>> transifex.egg-info/not-zip-safe | 1
>> transifex.egg-info/paster_plugins.txt | 2
>> transifex.egg-info/requires.txt | 2
>> transifex.egg-info/top_level.txt | 1
>
> Testing building/installing on clean systems needed. Eg.
>
> python setup.py sdist, bdist, bdist_egg, devel, install
> rm -rf dist && python setup.py sdist && cd dist && tar zxf
> *.tar.gz && cd transifex-0.3 && python setup.py install

Cool!

I have some questions though!

* Is there some problem with extra_requires?
* I think that the version attribute in setup.py should be `dev` or
something all the time. We should change dev with version numbers only
in the version release commit.
* There is some trailing whitespace in the patch: sed -i 's/[
\t]*$//' f-pypi-packaging.patch

Christos Τrochalakis

unread,
Jul 29, 2008, 4:00:02 AM7/29/08
to transif...@googlegroups.com
On Tue, Jul 29, 2008 at 10:58 AM, Christos Τrochalakis
<yat...@ideopolis.gr> wrote:
> * There is some trailing whitespace in the patch: sed -i 's/[
> \t]*$//' f-pypi-packaging.patch
>

Broken! Lets write it again:

Dimitris Glezos

unread,
Aug 2, 2008, 8:34:35 PM8/2/08
to transif...@googlegroups.com
On Tue, Jul 29, 2008 at 10:58 AM, Christos Τrochalakis
<yat...@ideopolis.gr> wrote:
> Cool!
>
> I have some questions though!
>
> * Is there some problem with extra_requires?

It adds complexity and I haven't seen any other projects use it
constructively. Open to any patches which add support for it in a
user-friendly way (eg. check if syntax highlighting was enabled in the
cfg file and then prompt the user to install `python-pygments`, etc).

> * I think that the version attribute in setup.py should be `dev` or
> something all the time. We should change dev with version numbers only
> in the version release commit.

Fixed and ReleaseNotes updated.

> * There is some trailing whitespace in the patch: sed -i 's/[
> \t]*$//' f-pypi-packaging.patch

I think it's fixed now.

Reply all
Reply to author
Forward
0 new messages