Sign me up Login

Details about package "wallpapoz"

Name: wallpapoz
Uploader: Daniel Echeverry <epsilon77@gmail.com> (Debian QA page)
Description: wallpapoz - GNOME Desktop Wallpapers Configuration Tool

Package versions

Version 0.6.1-1

Information

Version: 0.6.1-1
Uploaded: 2012-01-03 22:52
Source package: http://mentors.debian.net/debian/pool/main/w/wallpapoz/wallpapoz_0.6.1-1.dsc
Section: graphics
Priority: extra
Closes bugs: 648817

QA information

Info: /var/cache/debexpo/incoming/wallpapoz_0.6.1-1_source.changes
Info: wallpapoz_0.6.1-1.dsc
Info: Package is Lintian clean
Info: Package is not native
Info: The maintainer and uploader emails are the same
Info: Package is the latest upstream version
Info: A watch file is present
Info: The watch file works
Info: Closes WNPP bug #648817: "ITP: wallpapoz -- GNOME Desktop Wallpapers Configuration Tool"
Info: Homepage: http://vajrasky.wordpress.com/wallpapoz/
Info: The package uses straight debhelper

Comments

  1. I've looked through the package, and noted a few things that could do with improvement:
     * debian/control, debian/*.1:
       - Descriptions contain grammatical errors. http://paste.debian.net/151338/ is a slightly rephrased version of the long description in debian/control.
       - Translators in manpage includes &quot;My self&quot;.
    
     * debian/copyright:
       - GPL-2+, not GPL-2.0+
       - GFDL-1.1+, not GFDL-1.1 (license.xml states &quot;or later&quot;)
       - &quot;LIST THEIR TITLES&quot; is not meant to be included in the License: GFDL-1.1+ block.
       - GFDL is not DFSG-free (but in this case it is considered free due to an exception: https://wiki.debian.org/DFSGLicenses#GNU_Free_Documentation_License_.28FDL.29), which is why it is important to properly state that there are no invariant sections. You can get the GFDL snippet from the license.xml files.
       - Copyright information for wallpapoz.xml files are wrong. See the &lt;copyright&gt;...&lt;/copyright&gt; bits in the wallpapoz.xml files. The following command should help you fill that up:
    $ for i in **/wallpapoz.xml; do echo $i; sed -n '/&lt;copyright&gt;/,/&lt;\/copyright&gt;/p' $i; echo; done
    
     * debian/rules:
       - That's a lot of chmod statements there. You could replace pretty much all of them with &quot;find debian/wallpapoz -regex '\.(png|py|png|mo|xml)$$' -exec chmod 644 '{}' \+&quot;
       - Don't call dh_clean inside override_dh_auto_clean. dh(1) will take care of that. See the output of &quot;dh --no-act clean&quot;
    Needs work Chow Loong Jin at 2012-01-07 17:59:34.489016

  2. Hi,
    
    Thank you so much for your comments.
    
    Now I fixed the package, please check out
    http://debiancolombia.org/debian/wallpapoz/wallpapoz_0.6.1-1.dsc
    
    Thank You very much!!
    Daniel Echeverry at 2012-01-13 01:44:37.377830