Re: RFS: news (News - RSS Ticker)

2011-08-20 Thread Emmanuel Thomas-Maurin
On 08/20/2011 02:40 AM, Russ Allbery wrote:
 I think you need to use a bit less generic of a name than that.  (Like,
 for instance, news-rss-ticker, which would be a fine name).

Thank you. I've been looking for a not-too-long, easy-to-remember name
for binary/command, so (it may be a silly question) can package name and
binary name be different - package name: 'news-rss-ticker' and
binary/command: 'news'?

Or 'newsrt', 'nrt', 'nrticker', 'inews', 'i-news', 'gnews', ...?

-- 
Emmanuel Thomas-Maurin  manutm...@gmail.com


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e4f78ea.8070...@gmail.com



Re: RFS: news (News - RSS Ticker)

2011-08-20 Thread Emmanuel Thomas-Maurin
On 08/20/2011 02:37 AM, Vincent Cheng wrote:
 I'm not a DD so I can't sponsor your package, but I hope that the
 following comments and suggestions can help you improve your package.

Thank you. I'm working on fixing all these issues.

-- 
Emmanuel Thomas-Maurin  manutm...@gmail.com


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e4f7996.4050...@gmail.com



Re: RFS: news (News - RSS Ticker)

2011-08-20 Thread Arno Töll
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

On 20.08.2011 11:05, Emmanuel Thomas-Maurin wrote:
 Thank you. I've been looking for a not-too-long, easy-to-remember name
 for binary/command, so (it may be a silly question) can package name and
 binary name be different - package name: 'news-rss-ticker' and
 binary/command: 'news'?

yes, that's certainly possible, although we as distributor tend to
refrain from such changes and let upstream decide how to call their
programs. As you are upstream yourself, you might be able just to rename it.

The problem with /usr/bin/news is its generic name in a globally shared
name space. In fact there is a already /usr/bin/news in the package
sysnews making yours providing it as well a policy violation. Please see
Debian policy § 10.1 [1] in general and examples like [2][3] for a
little more background.



[1] http://www.debian.org/doc/debian-policy/ch-files.html#s-binaries
[2] http://lists.debian.org/debian-devel/2011/02/msg1.html
[3] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=611698

- -- 
with kind regards,
Arno Töll
IRC: daemonkeeper on Freenode/OFTC
GnuPG Key-ID: 0x9D80F36D
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJOT3rDAAoJEMcrUe6dgPNttGkQAKBsTl9hD10hfr/yW7153sdQ
v06Ne4Y56gjNO69/7Ebg4jLzzryec7dUwr02E0C3jaGD/K+kY7W2CpU7nmi6KJdN
PMh7S+Yz1090oCfc93aCR7wz3Ni44JpiPPmiMKTVy8J+eL8MpAIWMrddECELYJtr
tpEYrmisf7Q4HpVTgoQFpnbszOGBKnmecb7bYScqkjUiVpMH/bVp7ibXcbh5sc4a
wDQU3BGsFitwLq/FOleQSYyFacqaHZmrZza2mwSMUHw8uxm7vFhzyjFqwkl8E0cF
maO4gE89bBD1BlR4dHSsOOZXWX7dYtD1c/gFpjDlZkHFXlBGdyWLaGlmJzhdmjXk
S9XUf6dCVyuhUS5bMgT7TC5bKqkR+jOWsbCxsCwjga3bUitI4ZGhDfetaT29qYEA
7lilFaA4pxrZn0pNIId+VsQm0St5i9zYAZQe3s5kFO5nTQlhL0OBCElwcel/JTcw
cQaQLn5D0P3QPh8ODUN5WDzFyAqD6NeOGaGHtDMkiDyJY03QfObdsywd3JZfc2vq
YYLZNafs/+nYE0tuEbD/Q/mIIiGK4X+iGUJzu11yHcIXg7IA7hg5psIhNKqs5BRY
Dvtki5eFiH2ENlHpRrhs5JC2FksAJqjo69gF4jtUK2noKFiycW3ox2+BALFBlpEf
Xg1Fhvvp6lnQ6Y99tri4
=a2Ia
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4e4f7ac4.8050...@toell.net



Re: RFS: news (News - RSS Ticker)

2011-08-20 Thread Nick Leverton
On Sat, Aug 20, 2011 at 11:05:46AM +0200, Emmanuel Thomas-Maurin wrote:
 On 08/20/2011 02:40 AM, Russ Allbery wrote:
  I think you need to use a bit less generic of a name than that.  (Like,
  for instance, news-rss-ticker, which would be a fine name).
 
 Thank you. I've been looking for a not-too-long, easy-to-remember name
 for binary/command, so (it may be a silly question) can package name and
 binary name be different - package name: 'news-rss-ticker' and
 binary/command: 'news'?
 
 Or 'newsrt', 'nrt', 'nrticker', 'inews', 'i-news', 'gnews', ...?

inews and gnews already exist ...

Nick


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110820095528.ga18...@leverton.org



Re: RFS: news (News - RSS Ticker)

2011-08-19 Thread Vincent Cheng
On Fri, Aug 19, 2011 at 4:49 PM, Emmanuel Thomas-Maurin
manutm...@gmail.com wrote:
 Dear mentors,

 I am looking for a sponsor for my package news.

  * Package name    : news
   Version         : 0.5.3-1
   Upstream Author : Emmanuel Thomas-Maurin manutm...@gmail.com
  * URL             : http://www.newsrssticker.com/unstable-debian/
  * License         : GPL
   Section         : net

 To access further information about this package, please visit the
 following URL:

  http://mentors.debian.net/package/news

 Alternatively, one can download the package with dget using this command:

  dget -x http://mentors.debian.net/debian/pool/main/n/news/news_0.5.3-1.dsc

 I would be glad if someone uploaded this package for me.

 Kind regards,

 --
 Emmanuel Thomas-Maurin  manutm...@gmail.com


 --
 To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
 with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
 Archive: http://lists.debian.org/4e4ef698.7000...@gmail.com



Hi,

I'm not a DD so I can't sponsor your package, but I hope that the
following comments and suggestions can help you improve your package.

First of all, there's a load of lintian errors/warnings you should fix
(please aim for a lintian clean package before sending out a RFS
request). These can be identified by running lintian -I --pedantic
*.changes (also use -i if you want the extended description for these
lintian tags).

W: news source: configure-generated-file-in-source config.status
W: news source: configure-generated-file-in-source config.log
P: news source: source-contains-prebuilt-windows-binary dlls/libxml2-2.dll
W: news source: debhelper-overrides-need-versioned-build-depends (= 7.0.50~)
I: news source: missing-debian-source-format
W: news source: out-of-date-standards-version 3.8.3 (current is 3.9.2)
E: news: no-copyright-file
I: news: unknown-field-in-control screenshot-url
I: news: unknown-field-in-control appname
I: news: unknown-field-in-control category
I: news: unknown-field-in-control icon
I: news: unknown-field-in-control thumbnail-url
W: news: binary-without-manpage usr/bin/news
W: news: menu-item-creates-new-section Applications/Network/Web
usr/share/menu/news:8
W: news: executable-not-elf-or-script usr/share/news/doc/README

- Consider updating your source format to 3.0 (quilt)
- Bump your standards version after taking a look at the upgrade checklist. [1]
- You use various dh overrides, so bump your debhelper build
dependency to = 7.0.50~.
- Why do you override dh_installdocs and dh_installchangelogs? Instead
of manually installing them with debian/install, list your doc files
in debian/docs and let dh install them to the right directories.
- debian/watch is slightly broken:

$ uscan --report-status
dpkg: error: version 'unstable/news-0.5.3' has bad syntax: version
number does not start with digit
Processing watchfile line for package news...
Newest version on remote site is unstable/news-0.5.3, local version is 0.5.3
dpkg: error: version 'unstable/news-0.5.3' has bad syntax: version
number does not start with digit
news: Newer version (unstable/news-0.5.3) available on remote site:
  http://www.newsrssticker.com/unstable/news-0.5.3.tar.gz
  (local version is 0.5.3)

- Why does your binary package explicitly depend on libgtk2.0-0 and
libxml2 in debian/control? dh_shlibdeps should do the work for you and
automatically detect the right dependencies.
- You have various XB-* fields in debian/control that certainly aren't
standard on Debian (I don't know about Ubuntu though). I'm not sure if
they should be removed or not, but dpkg-deb does warn you about them
during the build process:

dpkg-deb: warning: 'debian/news/DEBIAN/control' contains user-defined
field 'Appname'
dpkg-deb: warning: 'debian/news/DEBIAN/control' contains user-defined
field 'Category'
dpkg-deb: warning: 'debian/news/DEBIAN/control' contains user-defined
field 'Icon'
dpkg-deb: warning: 'debian/news/DEBIAN/control' contains user-defined
field 'Screenshot-Url'
dpkg-deb: warning: 'debian/news/DEBIAN/control' contains user-defined
field 'Thumbnail-Url'
dpkg-deb: warning: ignoring 5 warnings about the control file(s)

- You need to write a manual page for your program. [2] and
man-pages(7) should get you started if you don't know how.
- Consider updating debian/copyright to comply with DEP-5. [3]
- dpkg-shlibdeps throws out a long list of warnings about uselessly
linked libraries. Consider fixing this.

Feel free to send out another RFS once you've fixed most of the above
issues, or ask if you don't quite understand. Good luck!

Kind regards,
- Vincent Cheng

[1] http://www.debian.org/doc/packaging-manuals/upgrading-checklist.txt
[2] http://liw.fi/manpages/
[3] http://dep.debian.net/deps/dep5/


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: