Merge lp:~james-w/pkgme/less-mandatory-information into lp:pkgme

[James Westby]

Hi,

Here's a small branch to reduce the number of required info elements.

Most things can have a default that will work, even if it's not useful information
(e.g. maintainer.) This will make it easier for backend authors to get started.

The only remaining required element is the package name. That could be given
a default of "package" or something, but I think that's going a bit far.

Thanks,

James

[Barry Warsaw (barry)]

What do you think about taking the email default from email.utils.formataddr() on the DEBEMAIL and DEBFULLNAME envars if they exist? Perhaps fall back to $NAME for the latter?

[James Westby (james-w)]

On Mon, 24 Jan 2011 19:20:09 -0000, Barry Warsaw <email address hidden> wrote:
> What do you think about taking the email default from
> email.utils.formataddr() on the DEBEMAIL and DEBFULLNAME envars if
> they exist? Perhaps fall back to $NAME for the latter?

I think that makes sense.

There's code in bzr-builder that actually implements the same as "dch"
does for this, which is probably the right way to go.

How do you feel about adding that later?

Thanks,

James

[Barry Warsaw (barry)]

On Jan 24, 2011, at 07:31 PM, James Westby wrote:

>On Mon, 24 Jan 2011 19:20:09 -0000, Barry Warsaw <email address hidden> wrote:
>> What do you think about taking the email default from
>> email.utils.formataddr() on the DEBEMAIL and DEBFULLNAME envars if
>> they exist? Perhaps fall back to $NAME for the latter?
>
>I think that makes sense.
>
>There's code in bzr-builder that actually implements the same as "dch"
>does for this, which is probably the right way to go.
>
>How do you feel about adding that later?

Sounds good. Probably should open a bug report on that for tracking
purposes. Then commit this branch as is.

=== modified file 'pkgme/info_elements.py'
--- pkgme/info_elements.py	2010-12-17 21:19:36 +0000
+++ pkgme/info_elements.py	2011-01-16 19:12:54 +0000
@@ -64,7 +64,7 @@
         Name <email>
 
     """
-    required = True
+    default = "A Person <[e-mail]>"
 
 
 class Homepage(InfoElement):
@@ -98,7 +98,7 @@
         to build on specific architectures only.
 
     """
-    required = True
+    default = "any"
 
 
 class Depends(InfoElement):
@@ -111,7 +111,7 @@
 
     name = "description"
     description = "The description of the package."
-    required = True
+    default = "a package"
 
 
 class Version(InfoElement):