Re: RFS: aoeui (updated package)
Hello Ryan, On Sat, Nov 22, 2008 at 02:22, Ryan K. [EMAIL PROTECTED] wrote: Updated package available on mentors, It would have been better it you went thru my list of points and commented each, if done or not (and why) since some are missing :) like, - some changes missing in changelog - double empty line between entry and tailer line of the last changelog entry - license (GPLv2) not clear stated for upstream work (when referring to local license file) - license for your work not clearly states the version of GPL but here come the worst part, it FTBFS in pbuilder (attached the full log): install -d /tmp/buildd/aoeui-1.2/debian/aoeui/usr/share/aoeui install -d /tmp/buildd/aoeui-1.2/debian/aoeui/usr/share/man/man1 install aoeui /tmp/buildd/aoeui-1.2/debian/aoeui/usr/bin ln -nf /tmp/buildd/aoeui-1.2/debian/aoeui/usr/bin/aoeui /tmp/buildd/aoeui-1.2/debian/aoeui/usr/bin/asdfg install *.1.gz /tmp/buildd/aoeui-1.2/debian/aoeui/usr/share/man/man1 make[1]: Leaving directory `/tmp/buildd/aoeui-1.2' dh_installdirs -paoeui find debian -type f -name *.txt | xargs chmod 644 chmod: missing operand after `644' Try `chmod --help' for more information. make: *** [install/aoeui] Error 123 dpkg-buildpackage: failure: fakeroot debian/rules binary gave error exit status 2 pbuilder: Failed autobuilding of package Please always test the build process in pbuilder, fix this and the other mentioned things, and reupload it. Kindly, -- Sandro Tosi (aka morph, Morpheus, matrixhasu) My website: http://matrixhasu.altervista.org/ Me at Debian: http://wiki.debian.org/SandroTosi aoeui_1.2-1_i386.build Description: Binary data
sizeof(int)
Is int 32 bit on all ports of debian? if not what ports is it not 32 bit on? more generally is there a list availible anywhere of debian architectures and the size of the standard C data types on them? -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: sizeof(int)
On Sat, Nov 22, 2008 at 14:46, peter green [EMAIL PROTECTED] wrote: Is int 32 bit on all ports of debian? if not what ports is it not 32 bit on? more generally is there a list availible anywhere of debian architectures and the size of the standard C data types on them? http://wiki.debian.org/ArchitectureSpecificsMemo Cheers, -- Sandro Tosi (aka morph, Morpheus, matrixhasu) My website: http://matrixhasu.altervista.org/ Me at Debian: http://wiki.debian.org/SandroTosi -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: sizeof(int)
On Saturday 22 November 2008 14:46:44 peter green wrote: Is int 32 bit on all ports of debian? if not what ports is it not 32 bit on? int is not definied as 32 bit use uint32_t (from stdint.h) for that. Please look at the specific abi specifiction of your plattform. And you can look at http://wiki.debian.org/ArchitectureSpecificsMemo for possible differences. -- Robert Wohlrab -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: RFS: ocropus (3nd try)
Hello Jeffrey, On Mon, Nov 10, 2008 at 14:06, Sandro Tosi [EMAIL PROTECTED] wrote: On Sun, Nov 9, 2008 at 22:01, Jeffrey Ratcliffe [EMAIL PROTECTED] wrote: I am looking for a sponsor for my package ocropus. I'm giving the package a look, and here are my comments: Did you find the time (or already planned to do so) to fix the points I mentioned? It would be really nice to upload it asap, since it seems a really outstanding app :) Kindly, -- Sandro Tosi (aka morph, Morpheus, matrixhasu) My website: http://matrixhasu.altervista.org/ Me at Debian: http://wiki.debian.org/SandroTosi -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
Uploaded an updated package to http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-4.dsc Thanks in advance for your feedback. -- Jonathan Wiltshire signature.asc Description: Digital signature
RFS: latexdiff (take 2)
Hello dear mentors. I've packaged latexdiff (ITP:#469460), a small utilty to generate friendly latex file that contains the annotated diff of the versions of the same document. Very useful for non technical ppl. You can access the git repo here: http://git.debian.org/?p=users/munga-guest/latexdiff.git;a=summary (I use git-buildpackage) or otherwise you can get the package here with dget : dget -u http://alioth.debian.org/~munga-guest/latexdiff_0.5-1.dsc (it's not signed. I don't have my keys handy atm) I think a good place for this package would be in the texlive group on alioth... Thanks martin for the review. :) p -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED] -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: RFS: latexdiff (take 2)
Hello Pietro, On Sat, Nov 22, 2008 at 17:52, Pietro Abate [EMAIL PROTECTED] wrote: Hello dear mentors. I've packaged latexdiff (ITP:#469460), a small utilty to generate friendly latex file that contains the annotated diff of the versions of the same document. Very useful for non technical ppl. You can access the git repo here: http://git.debian.org/?p=users/munga-guest/latexdiff.git;a=summary (I use git-buildpackage) or otherwise you can get the package here with dget : dget -u http://alioth.debian.org/~munga-guest/latexdiff_0.5-1.dsc (it's not signed. I don't have my keys handy atm) I think a good place for this package would be in the texlive group on alioth... So let's give them a chance to see this email (I add in CC) :) Thanks martin for the review. I'd be willing to check it (hopefully sponsoring too) if others don't have time to :) Regards, -- Sandro Tosi (aka morph, Morpheus, matrixhasu) My website: http://matrixhasu.altervista.org/ Me at Debian: http://wiki.debian.org/SandroTosi -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: debconf questions
On 10/11/2008, Adeodato Simó wrote: Hm, I thought you're not supposed to use debconf to display notes, and that NEWS.Debian is preferred. Yeah, otherwise Christian is coming after you; and you don't want that. Mraw, KiBi. signature.asc Description: Digital signature
Re: RFS: ocropus (3nd try)
2008/11/22 Sandro Tosi [EMAIL PROTECTED]: Did you find the time (or already planned to do so) to fix the points I mentioned? I've got about half way through them. Real Life is just making me short of time ATM. I'll get to the rest ASAP. Regards Jeff -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: Re: RFS: xinha
Hi,
debian/control:
What about adding a replaces: htmlarea? or maybe some other mention so that
people looking for htmlarea actually find something (better).
Build-Depends: debhelper (= 6.0.7~)
what is that tilde doing there? (lintian should probably warn about it)
debian/rules:
(cleanup) build-stamp is not needed, you can get rid of it ('build' is a PHONY
target anyway).
#marking files as not executable (lintian warnings)
find debian/xinha/usr/share/xinha/www -type f -exec chmod -x '{}' \;
find debian/xinha/usr/share/doc/xinha/examples -type f -exec chmod -x '{}'
\;
chmod -R -x,+X debian/xinha/usr/share/xinha/www
chmod -R -x,+X debian/xinha/usr/doc/xinha/examples
cp contrib/php-xinha.php $(CURDIR)/debian/xinha/usr/share/doc/xinha/examples
dh_installchangelogs -i release-notes.txt
Why are these in binary-indep and not in install just like the rest?
(I know must packages do something similar, even some (if not all) of mine).
debian/copyright-REMOVED:
nice work :)
debian/uupdate-wrapper:
| gzip xinha_$version.orig.tar.gz
use max compression (a.k.a -9)
You should also:
* use 'set -e'
* better use the same file name as in $file (there should be no worries about
replacing the file while reading from it because tar requires the whole tarball
to be available/read).
debian/copyright:
Format-Specification:
http://wiki.debian.org/Proposals/CopyrightFormat?action=recallrev=REVISION
You are actually supposed to replace REVISION with the revision of the page you
based your copyright file on.
debian/examples:
I can see some references to the php-xinha.php file in xinha itself, are you
sure it should be installed just as an example?
Cheers,
--
Raphael Geissert - Debian Maintainer
www.debian.org - get.debian.net
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: RFS: xinha
Raphael Geissert [EMAIL PROTECTED] writes: Build-Depends: debhelper (= 6.0.7~) what is that tilde doing there? (lintian should probably warn about it) It allows the package to build with backports of debhelper 6.0.7. -- Russ Allbery ([EMAIL PROTECTED]) http://www.eyrie.org/~eagle/ -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
On Sun, Nov 23, 2008 at 1:30 AM, Jonathan Wiltshire
[EMAIL PROTECTED] wrote:
http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-4.dsc
More comments:
It is best not to speculate about future features in the package
description, please remove the last line.
Uhhh, Depends: is a header for binary packages, not source packages.
You put ${misc:Depends} in the wrong section:
dpkg-gencontrol: warning: unknown information field 'Depends' in input
data in general section of control info file
debian/rules configure doesn't seem to be needed, delete it and remove
it from .PHONY.
The upstream code does not contain any copyright information, please
ask upstream to fix it (add add your manual page at the same time).
Please also ask upstream to add the standard GPL license grant to the
script so that there can be no confusion about which version of the
GPL is to be used.
--
bye,
pabs
http://wiki.debian.org/PaulWise
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
