Merge lp://staging/~jml/pkgme/default-maintainer into lp://staging/pkgme

Proposed by Jonathan Lange
Status: Merged
Approved by: James Westby
Approved revision: 100
Merged at revision: 101
Proposed branch: lp://staging/~jml/pkgme/default-maintainer
Merge into: lp://staging/pkgme
Diff against target: 63 lines (+21/-0)
2 files modified
pkgme/info_elements.py (+6/-0)
pkgme/tests/test_info_elements.py (+15/-0)
To merge this branch: bzr merge lp://staging/~jml/pkgme/default-maintainer
Reviewer Review Type Date Requested Status
James Westby Approve
Review via email: mp+99299@code.staging.launchpad.net

Commit message

Use DEBEMAIL environment variable to set the package maintainer.

Description of the change

This changes the Maintainer field to use the DEBEMAIL environment variable if it's set. Otherwise, uses the dummy "A Person" email address that pkgme currently uses.

The goal of this change is to allow pkgme-service to specify the maintainer for the packages that it builds.

To post a comment you must log in.
Revision history for this message
Jonathan Lange (jml) wrote :

Note that the python_debian on my precise system seems to have a get_maintainer() function that we could & probably should use, but that this function is not available in my pkgme virtualenv.

Revision history for this message
James Westby (james-w) wrote :

The algorithm for DEBEMAIL and friends is more complex than this, but I think
this is an ok start. Using the get_maintainer() function at a later date would
be good.

Thanks,

James

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches