PQM

Doesn't use setup.py

Bug #117197 reported by Jelmer Vernooij
2
Affects Status Importance Assigned to Milestone
PQM
Won't Fix
High
Unassigned

Bug Description

  affects /products/pqm

The makefile only installs the binary but not the required 'pqm' python
package.

Is there any reason for using the auto* tools rather than distutils ?

Related branches

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 117197] Doesn't install required Python package

On Sun, 2007-05-27 at 14:33 +0000, Jelmer Vernooij wrote:
> Public bug reported:
>
> affects /products/pqm
>
> The makefile only installs the binary but not the required 'pqm' python
> package.

I've always run PQM in-place.

> Is there any reason for using the auto* tools rather than distutils ?

Hysterical raisons.

-Rob
--
GPG key available at: <http://www.robertcollins.net/keys.txt>.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Robert Collins wrote:
> On Sun, 2007-05-27 at 14:33 +0000, Jelmer Vernooij wrote:
>> Is there any reason for using the auto* tools rather than distutils ?
> Hysterical raisons.
Hmm.. so patches to change that situation would be welcome ?

Cheers,

Jelmer

P.S. Did you see my followup to your review of my other pqm-bzr patch?

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQCVAwUBRl6Wfgy0JeEGD2blAQLBowP9E0foXdq34rbpPjrtCIZ5YM91X9pMk0u5
Gy7lyLZWBnc1A7Br4DYhlff0BroLCyNYnD58Fo8/Xdc4gSsjoZCP7k+BSbE7IdgN
q71fH9tutknWrXmPwkIRSySQNEMyr/jBG8XDOEAoCuRAzsPQ0jVZxEb9MsxZB73V
SsRn3GCoAt0=
=plVu
-----END PGP SIGNATURE-----

Revision history for this message
Robert Collins (lifeless) wrote :

On Thu, 2007-05-31 at 09:33 +0000, Jelmer Vernooij wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Robert Collins wrote:
> > On Sun, 2007-05-27 at 14:33 +0000, Jelmer Vernooij wrote:
> >> Is there any reason for using the auto* tools rather than distutils ?
> > Hysterical raisons.
> Hmm.. so patches to change that situation would be welcome ?

Hmmm, if they are a net win, sure. Theres a bunch of stuff that
setuptools is no better at managing - see bzr's use of a plain Makefile
for instance.

-Rob
--
GPG key available at: <http://www.robertcollins.net/keys.txt>.

Revision history for this message
Robert Collins (lifeless) wrote : Re: Doesn't install required Python package

Rather important to make it packagable.

Changed in pqm:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Jelmer Vernooij (jelmer) wrote :
Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: [Bug 117197] Re: Doesn't install required Python package

  status fixcommitted

Fixed in the attached bundle.
--
Jelmer Vernooij <email address hidden> - http://samba.org/~jelmer/
Jabber: <email address hidden>

Changed in pqm:
status: Triaged → Fix Committed
Revision history for this message
Robert Collins (lifeless) wrote : Re: Doesn't install required Python package

So I've looked at the patch and it does not seem to be a net win to me. On reflection I'd really rather keep just fix the failure to install the package and not convert everything to setup.py - setup.py doesn't seem simpler to me.

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

automake is a very heavy dependency for a simple python package such as pqm.

We could still ditch autoconf and automake and use a simple Makefile for building the docbook stuff?

Revision history for this message
Tim Penhey (thumper) wrote :

Given that PQM is primarily written in python, I'd have to argue that a setup.py is much easier to understand for the majority of people that would be looking to work with the code. By keeping autoconf and automake you are scaring away potential contributors.

I'd vote for moving to setup.py.

Revision history for this message
Dan Watkins (oddbloke) wrote :

I'd prefer a setup.py also. I'm simply not familiar with autotools.

Revision history for this message
Jelmer Vernooij (jelmer) wrote : [MERGE] Updated distutils patch

Updated version of my patch against current trunk.

Cheers,

Jelmer

--

Revision history for this message
Stuart Bishop (stub) wrote : Re: Doesn't install required Python package

Resetting the status - this isn't in the trunk yet.

The net win includes familiarity to Python developers, ability to distribute via egg on the Python package index, platform independence, Python version independence, and that nobody will need to write the INSTALL document.

It took a few minutes for me to work out the undocumented procedure on how to generate and run the Makefile (autogen.sh?) that gives me a big autogenerated Makefile that I certainly can't be arsed decoding to work out what the rules are and what they do.

Changed in pqm:
status: Fix Committed → New
Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 117197] Re: Doesn't install required Python package

 status fix-committed

It is committed in a branch; pqm uses the same status values as bzr;
committed -> fix-committed
on trunk -> fix-released

Revision history for this message
Robert Collins (lifeless) wrote :

 status fixcommitted

It is committed in a branch; pqm uses the same status values as bzr;
committed -> fix-committed
on trunk -> fix-released

Changed in pqm:
status: New → Fix Committed
Changed in pqm:
status: Fix Committed → Won't Fix
summary: - Doesn't install required Python package
+ Doesn't use setup.py
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.