Installs from source distributions fail

Bug #735284 reported by Chris Adams
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Meliae
Fix Released
Medium
Chris Adams

Bug Description

lp:~acdha/meliae/setup.py-fix contains a MANIFEST.in file which includes the .h and .c files needed to actually compile the extension when working from an sdist package rather than a source checkout.

Related branches

Revision history for this message
John A Meinel (jameinel) wrote :

Can you explain this a bit more? Are you asking for me to merge that branch?

Revision history for this message
Chris Adams (acdha) wrote : Re: [Bug 735284] Re: Installs from source distributions fail

Last night I found that a pip / easy_install of meliae from PyPI fails because it needs _scanner.c and _scanner_core.h, which aren't in the tarball. This fixes that in my limited testing but I didn't send a pull request because I wasn't sure whether you have a different, preferred way to package this.

Chris
On Tuesday, March 15, 2011 at 7:07 AM, John A Meinel wrote:
> Can you explain this a bit more? Are you asking for me to merge that
> branch?
>
> --
> You received this bug notification because you are a direct subscriber
> of the bug.
> https://bugs.launchpad.net/bugs/735284
>
> Title:
> Installs from source distributions fail
>
> Status in Meliae python memory analysis:
> New
>
> Bug description:
> lp:~acdha/meliae/setup.py-fix contains a MANIFEST.in file which
> includes the .h and .c files needed to actually compile the extension
> when working from an sdist package rather than a source checkout.
>
> To unsubscribe from this bug, go to:
> https://bugs.launchpad.net/meliae/+bug/735284/+subscribe
>

Revision history for this message
John A Meinel (jameinel) wrote :

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

On 3/15/2011 4:18 PM, Chris Adams wrote:
> Last night I found that a pip / easy_install of meliae from PyPI fails
> because it needs _scanner.c and _scanner_core.h, which aren't in the
> tarball. This fixes that in my limited testing but I didn't send a pull
> request because I wasn't sure whether you have a different, preferred
> way to package this.

Those certainly should be included in the tarball. Some of the other
header and C files are auto-generated, but those are hand-crafted.

Feel free to propose it, so I can get a look at it.

John
=:->

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

iEYEARECAAYFAk2AFMMACgkQJdeBCYSNAAMFUQCdEuFwEwU9N65dinWTalo7hbyr
hGMAoMj0yD/fiCtwn+pF393JAD/Zet52
=ktbg
-----END PGP SIGNATURE-----

Revision history for this message
dan kuebrich (dan-kuebrich) wrote :

+1 same problem

John A Meinel (jameinel)
Changed in meliae:
status: New → Fix Released
assignee: nobody → Chris Adams (acdha)
importance: Undecided → Medium
John A Meinel (jameinel)
Changed in meliae:
milestone: none → 0.4
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.