RFS: free42

2011-01-06 Thread Jean Schurger
Dear mentors,

I am looking for a sponsor for my package free42.

* Package name: free42
  Version : 1.4.66-1
  Upstream Author : Thomas Okken
* URL : http://thomasokken.com/free42/
* License : GPL2
  Section : misc

It builds these binary packages:
free42 - HP42S Emulator

The package appears to be lintian clean.

My motivation for maintaining this package is:
This software maintained by its author and really usefull,
and it's a free implementation.

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/f/free42
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

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

Kind regards
Jean Schurger
-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc





signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2011-01-06 Thread Bernhard R. Link
* Jean Schurger j...@schurger.org [110106 15:40]:
 I am looking for a sponsor for my package free42.

Looking at debian/changelog, that is not quite helpful:

| free42 (1.4.66-1) unstable; urgency=low
|
|  * Upstream release
|  * Closes: #606358
|
| -- Jean Schurger j...@schurger.org  Wed, 08 Dec 2010 12:38:31 -0500

The Closes has no reference to anything (so one could guess that it is the ITP
that is closed). And what is Upstream release as changelog for the very first
version supposed to tell me?

Next debian/control:

| Source: free42
| Section: misc
| Priority: optional
| Homepage: http://thomasokken.com/free42/
| Maintainer: Jean Schurger j...@schurger.org
| Build-Depends: debhelper (= 7.0.50), libxmu-dev, libgtk2.0-dev (= 2.10.3)
| Standards-Version: 3.9.1
|
| Package: free42
| Architecture: any
| Depends: ${misc:Depends}, ${shlibs:Depends}, debconf | debconf-2.0
| Description: HP42S Emulator
|  Free42 is a re-implementation of the HP-42S calculator and the HP-82240
|  printer. It is a complete rewrite, and doesn't use any HP code.
|  You do not need an HP-42S ROM image in order to use it,
|  yet it is fully HP-42S compatible.
|  .

There is no need for an empty line at the end, so you can remove it
(the one with the dot).
I guess section otherosfs would be better than misc, as most other emulators
are there.

The It is a complete rewrite, and doesn't use any HP code. is a bit strange
to be in a description. (If it is not free software, it does not belong into
Debian. If it is free software, the copyright holder is not that important).

Why is this thing written without - in the short and with - in the long
description? I also think  calculator could be added to the short description
and the long description be a bit more verbose what this thing is it emulates
might be helpfull.

Your debian/copyright file only lists
Thomas Okken and you, but as simple grep -i -r copyright . already gives:

Copyright _ 2005, Apple Computer, Inc.  All rights reserved.
Copyright 2005 D. Jeff Dionne
Copyright (C) 1997 Rick Huebner
Copyright (C) 1993 by Sun Microsystems, Inc. All rights reserved.
Copyright (C) 1997 Free Software Foundation, Inc.
Copyright (c) 1998 John D. Polstra.
Copyright (c) 2004-2005 Greg Parker.  All rights reserved.
Copyright (c) 2001 David E. O'Brien
Copyright (c) 2005-2009 voidware ltd.
Copyright Base2 Corporation 2009
Copyright (C) 1997 Rick Huebner

Some of which seems to even be compiled and explicitly have BSD like licenses
requiring to include their copyright statement (which your binary packages do 
not).

I've not looked at the actual packaging or the build process.

Bernhard R. Link



-- 
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/20110106152254.ga16...@pcpool00.mathematik.uni-freiburg.de



Re: RFS: free42

2010-12-24 Thread Osamu Aoki
Hi,

On Tue, Dec 21, 2010 at 11:35:14AM -0500, Jean Schurger wrote:
 Le mardi 21 décembre 2010 à 11:25 +0100, Etienne Millon a écrit :
  On Mon, Dec 20, 2010 at 09:49:31AM -0500, Jean Schurger wrote:
   I would like to simplify the debian/rules using (dh $@) but i think
   i don't understand clearly what is mean. Upstream sources does not
   use autotools. Is (dh $@) still applicable ? I think i'm missing
   something.
  
  Yes, it can be applied. The idea is for example that the install
  target will do default actions (call make install, build temporary
  directories, compress files, build deb…) except when you override a
  part. Moreover, certain parts follow a certain template : create empty
  directories, move files, etc. So they can be moved out of the
  Makefile, in package.dirs, package.install files, and so on.
  
  The mailing list does not seem to allow attachments, so I sent you a
  patch set that does exactly that. I factored your debian/rules to do
  the minimum in it, and use debhelper for the rest (it might be
  possible to reduce it further with dh --sourcedirectory=gtk but I am
  not sure). I kept the patches separate so that you can follow the
  process.
  
  Oh, and I moved doc/free42/README.gtk to doc/free42/README as this is
  the general purpose readme file for this package.

You meant gtk/README to usr/share/doc/free42/README while not installing
README ?
 
 Oh nice, 
 
   Thanks for thoses explanations. Now i see much better what yo mean.
 (I've already seen package sources with install and dirs files, without
 really knowing why it was working).
 I'm surprised to see manpages links are done properly on gziped files
 without telling.
 (I've added a little line on the debian/copyright to notice you help :p)
 
 I've updated the package on
 http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

What I downloaded had some issues.  Your package contains:
 debian/patches/debian-changes-1.4.66-1
This may be due to non -clean source tree you used to build package.  I
think you should have removed ChangeLog file after you created it for
some reason when playing with the source.

I did not understand why debian/gtk/Makefile was there after you start
using quilt.

As for debian/rules, I wonder why you make override for dh_clean and
dh_install.  Aren't they meant for dh_auto_* targets?  Something like:
-
#!/usr/bin/make -f

%:
dh $@

override_dh_auto_build:
cd gtk  make
cd gtk  make BCD_MATH=1

override_dh_auto_clean:
cd gtk  make cleaner
rm -f gtk/free42.xpm

override_dh_auto_install:
cp gtk/icon.xpm gtk/free42.xpm

override_dh_installchangelogs:
dh_installchangelogs HISTORY


At least, these fix some irregularities I see.  (These are not tested
though ...) See more on my explanation at:
http://www.debian.org/doc/manuals/maint-guide/ch-dreq.en.html#s-defaultrules

As for --sourcedirectory thing, I see it explained in debhelper(7) under
BUILD SYSTEM OPTIONS.   I never packaged this kind of package but
looks like what you need is:

-
#!/usr/bin/make -f

%:
dh $@

override_dh_auto_build:
dh_auto_build --sourcedirectory=gtk
dh_auto_build --sourcedirectory=gtk -- BCD_MATH=1

override_dh_auto_clean:
rm -f gtk/free42.xpm
dh_auto_clean --sourcedirectory=gtk -- cleaner

override_dh_auto_install:
cp gtk/icon.xpm gtk/free42.xpm

override_dh_installchangelogs:
dh_installchangelogs HISTORY



As for debian/patches/0_less-libs.diff file, you should use DEP-3 format
http://dep.debian.net/deps/dep3/

As for copyright file, you should use DEP-5 format
http://dep.debian.net/deps/dep5/

I do not have time now to check and test these ...  so I may have some
wrong comments ...

Osamu


--
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/20101224154831.ga12...@debian.org



Re: RFS: free42

2010-12-24 Thread Jean Schurger
Le samedi 25 décembre 2010 à 00:48 +0900, Osamu Aoki a écrit :
 Hi,

   
   Oh, and I moved doc/free42/README.gtk to doc/free42/README as this is
   the general purpose readme file for this package.
 
 You meant gtk/README to usr/share/doc/free42/README while not installing
 README ?

Exactly what he meant.

 At least, these fix some irregularities I see.  (These are not tested
 though ...) See more on my explanation at:
 http://www.debian.org/doc/manuals/maint-guide/ch-dreq.en.html#s-defaultrules
 
 As for --sourcedirectory thing, I see it explained in debhelper(7) under
 BUILD SYSTEM OPTIONS.

I'm testing that.


 As for debian/patches/0_less-libs.diff file, you should use DEP-3 format
 http://dep.debian.net/deps/dep3/

Ok, i fix that.

 As for copyright file, you should use DEP-5 format
 http://dep.debian.net/deps/dep5/
 

I have a two question about that:

- Can you point me to an example of a package already using that, it
will help me a lot.
- Is there a way to validate the file, i mean, check that it's ok for
computer usage ?

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-21 Thread Etienne Millon
On Mon, Dec 20, 2010 at 09:49:31AM -0500, Jean Schurger wrote:
 I would like to simplify the debian/rules using (dh $@) but i think
 i don't understand clearly what is mean. Upstream sources does not
 use autotools. Is (dh $@) still applicable ? I think i'm missing
 something.

Yes, it can be applied. The idea is for example that the install
target will do default actions (call make install, build temporary
directories, compress files, build deb…) except when you override a
part. Moreover, certain parts follow a certain template : create empty
directories, move files, etc. So they can be moved out of the
Makefile, in package.dirs, package.install files, and so on.

The mailing list does not seem to allow attachments, so I sent you a
patch set that does exactly that. I factored your debian/rules to do
the minimum in it, and use debhelper for the rest (it might be
possible to reduce it further with dh --sourcedirectory=gtk but I am
not sure). I kept the patches separate so that you can follow the
process.

Oh, and I moved doc/free42/README.gtk to doc/free42/README as this is
the general purpose readme file for this package.

Cheers,

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-21 Thread Jean Schurger
Le mardi 21 décembre 2010 à 11:25 +0100, Etienne Millon a écrit :
 On Mon, Dec 20, 2010 at 09:49:31AM -0500, Jean Schurger wrote:
  I would like to simplify the debian/rules using (dh $@) but i think
  i don't understand clearly what is mean. Upstream sources does not
  use autotools. Is (dh $@) still applicable ? I think i'm missing
  something.
 
 Yes, it can be applied. The idea is for example that the install
 target will do default actions (call make install, build temporary
 directories, compress files, build deb…) except when you override a
 part. Moreover, certain parts follow a certain template : create empty
 directories, move files, etc. So they can be moved out of the
 Makefile, in package.dirs, package.install files, and so on.
 
 The mailing list does not seem to allow attachments, so I sent you a
 patch set that does exactly that. I factored your debian/rules to do
 the minimum in it, and use debhelper for the rest (it might be
 possible to reduce it further with dh --sourcedirectory=gtk but I am
 not sure). I kept the patches separate so that you can follow the
 process.
 
 Oh, and I moved doc/free42/README.gtk to doc/free42/README as this is
 the general purpose readme file for this package.

Oh nice, 

  Thanks for thoses explanations. Now i see much better what yo mean.
(I've already seen package sources with install and dirs files, without
really knowing why it was working).
I'm surprised to see manpages links are done properly on gziped files
without telling.
(I've added a little line on the debian/copyright to notice you help :p)

I've updated the package on
http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

Can you uploaded it to debian ?

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-21 Thread Etienne Millon
On Tue, Dec 21, 2010 at 11:35:14AM -0500, Jean Schurger wrote:
 Oh nice, 
 
   Thanks for thoses explanations. Now i see much better what yo mean.
 (I've already seen package sources with install and dirs files, without
 really knowing why it was working).

You're welcome. I actually learnt from this, too.

 I'm surprised to see manpages links are done properly on gziped files
 without telling.

It's done in the free42.links file (interpreted by dh_link, called
by dh install).

 Can you uploaded it to debian ?

Sorry, I am not a Debian Developer si I can't sponsor you. However
your package seems good to me. I may miss something, though.

Now you just have to be patient until a DD reviews your package and
decides that it is suitable for inclusion in Debian. Note that we're
in deep freeze and the priority is to fix release-critical bugs. But
after Squeeze is released (ie when it's ready) it will probably be
possible.

Cheers,

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-20 Thread Etienne Millon
On Sun, Dec 19, 2010 at 10:44:15AM -0500, Jean Schurger wrote:
 Hi, i've updated the packages, can you have a look ?
 
 http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc
 
 Jean.

Hello,

It seems good to me. It builds in a clean chroot, and is lintian clean
upto -E. debian/rules can probably be simplified with compat level 7
(dh $@) but works as is.

Minor nitpicks :

  - The quote from upstream in debian/copyright should probably be
wrapped.

  - In your manpage, I don't think that you have to put that much
whitespace (see attached patch).

Cheers,

-- 
Etienne Millon
diff -Nru a/debian/free42bin.1 b/debian/free42bin.1
--- a/debian/free42bin.1	2010-12-15 17:14:53.0 +0100
+++ b/debian/free42bin.1	2010-12-20 13:03:00.0 +0100
@@ -6,7 +6,6 @@
 .RB [ \-skin
 .I SKIN
 ]
-
 .SH DESCRIPTION
 Free42 is a complete re-implementation of the HP-42S scientific programmable
 RPN calculator.
@@ -17,15 +16,12 @@
 or
 .I Realistic
 look for the emulator.
-
-
 .SH DOCUMENTATION
 The ultimate documentation for Free42 is the manual for the HP-42S. You can
 obtain this manual in PDF format by purchasing the CD or DVD set from The
 Museum of HP Calculators (http://hpmuseum.org/). Alternatively, there is an
 independently written HP-42S/Free42 manual, by Jose Lauro Strapasson, which
 you can download free at http://joselauro.com/42s.pdf.
-
 .SH EDITIONS
 What's the deal with the Decimal and Binary (free42dec/free42bin)?
 
@@ -51,16 +47,13 @@
 you need full HP-42S compatibility, you should use Free42 Decimal.
 If you don't fully understand the above, it is best to play safe and use
 Free42 Decimal (free42dec).
-
 .SH SEE ALSO
 The original documentation provided upstream, available in
 /usr/share/doc/free42/README.gtk.gz
 The keymap in
 /usr/share/doc/free42/keymap.txt.gz
-
 .SH AUTHORS/CREDITS
 Free42 is (C) 2004-2010, by Thomas Okken
 BCD support (C) 2005-2009, by Hugh Steers / voidware
-
 .SH LICENSE
 GPL-2


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-20 Thread Jean Schurger
Le lundi 20 décembre 2010 à 13:07 +0100, Etienne Millon a écrit :
 On Sun, Dec 19, 2010 at 10:44:15AM -0500, Jean Schurger wrote:
  Hi, i've updated the packages, can you have a look ?
  
  http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc
  
  Jean.
 
 Hello,
 
 It seems good to me. It builds in a clean chroot, and is lintian clean
 upto -E. debian/rules can probably be simplified with compat level 7
 (dh $@) but works as is.
 
 Minor nitpicks :
 
   - The quote from upstream in debian/copyright should probably be
 wrapped.
 
   - In your manpage, I don't think that you have to put that much
 whitespace (see attached patch).

Hi,
Thanks for your patch. I've quoted the copyright quotes as suggested.

I would like to simplify the debian/rules using (dh $@) but i think i
don't understand clearly what is mean. Upstream sources does not use
autotools. Is (dh $@) still applicable ? I think i'm missing something.


Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-19 Thread Jean Schurger
Le vendredi 10 décembre 2010 à 20:02 +0100, Etienne Millon a écrit :
 Again, no need to CC me :)
 
  Oh yes, i did not see the second patch (I'm not sure how it appear,
  maybe when i was playing with quilt).
 
 According to the patch header, it has been created by dpkg-source. I
 can't remember the exact process, though.
 
  Well, there is only one option: -skin, I'll put it in the man page.
  But the upstream comes with other documentation than two README files
  that overlap. I will figure what to do with that. Might be an upstream
  issue too.
 
 Documenting this and give a pointer to the full documentation may be
 enough.
  
 - Probably an upstream issue, but I find it confusing to ship two
   binaries that differ only in the way they interpret numbers. IMHO
   that should be a command-line switch or a menu option.
  
  I've looked quickly to the sources, move that 'option' into a switch
  will be a very big patch, but i think debian have a solution for that.
  May be i can build two packages, and use update-alternatives to handle
  that problem. Is that an acceptable solution for you ?
 
 Alternatives makes sense when two packages provides the same service.
 With free42, different users may use different settings (or, a user
 may use different settings). Building two executables is fine. You can
 even have a single manpage (see for example grep, egrep, rgrep).
 
 Cheers
 

Hi, i've updated the packages, can you have a look ?

http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

Jean.


-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-15 Thread Jean Schurger
Le vendredi 10 décembre 2010 à 20:02 +0100, Etienne Millon a écrit :
 Again, no need to CC me :)
 
  Oh yes, i did not see the second patch (I'm not sure how it appear,
  maybe when i was playing with quilt).
 
 According to the patch header, it has been created by dpkg-source. I
 can't remember the exact process, though.
 
  Well, there is only one option: -skin, I'll put it in the man page.
  But the upstream comes with other documentation than two README files
  that overlap. I will figure what to do with that. Might be an upstream
  issue too.
 
 Documenting this and give a pointer to the full documentation may be
 enough.
  
 - Probably an upstream issue, but I find it confusing to ship two
   binaries that differ only in the way they interpret numbers. IMHO
   that should be a command-line switch or a menu option.
  
  I've looked quickly to the sources, move that 'option' into a switch
  will be a very big patch, but i think debian have a solution for that.
  May be i can build two packages, and use update-alternatives to handle
  that problem. Is that an acceptable solution for you ?
 
 Alternatives makes sense when two packages provides the same service.
 With free42, different users may use different settings (or, a user
 may use different settings). Building two executables is fine. You can
 even have a single manpage (see for example grep, egrep, rgrep).

Hi,
  I've fixed the man (and doc), and tuned the Makefile patch. Can you
have a look please ?

Jean. 

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-10 Thread Etienne Millon
On Thu, Dec 09, 2010 at 08:07:30PM -0500, Jean Schurger wrote:
 Le jeudi 09 décembre 2010 à 08:59 +0100, Etienne Millon a écrit :
  On Wed, Dec 08, 2010 at 05:39:25PM -0500, Jean Schurger wrote:
   What's the 'good' way to ask to review an update of a package like this
   one ? I should continue to dput it as replacement, and ask in that
   thread ?
  
  dput -f mentors it will overwrite the previous package. You can
  state that you updated it in this thread and it will probably be fine.
 
 
 Hi, i've updated my free42 package.
 
 The package can be found on mentors.debian.net:
 - URL: http://mentors.debian.net/debian/pool/main/f/free42
 - Source repository: deb-src http://mentors.debian.net/debian unstable
 main contrib non-free
 - dget
 http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc
 
 Can you have a look ?
 
 Jean.

Hello,

Package builds cleanly and is lintian clean. That's good news :)

  - dpkg_shlibdeps does not complain anymore. It means that your patch
works. However, you have included two patches, and one
(0_less_libs.diff) does not apply (probably because your output
directory was named 2). The second one is fine, though. You can
delete the non-working one and remove it from
debian/patches/series.

  - you use a lot of calls to pwd in debian/rules. This is not
necessary, relative paths work too : `pwd`/x is equivalent to x.
Moreover, debhelper can help a lot here (dh $@).

  - Your manpages should describe how the binary is run. Does it have
command-line options ? etc. That will be the first thing your
users will try if they don't get how the package work. Other
documentation should go… in the documentation :-) .

  - Probably an upstream issue, but I find it confusing to ship two
binaries that differ only in the way they interpret numbers. IMHO
that should be a command-line switch or a menu option.

Cheers

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-10 Thread Jean Schurger
Le vendredi 10 décembre 2010 à 10:53 +0100, Etienne Millon a écrit :
 On Thu, Dec 09, 2010 at 08:07:30PM -0500, Jean Schurger wrote:
  Le jeudi 09 décembre 2010 à 08:59 +0100, Etienne Millon a écrit :
   On Wed, Dec 08, 2010 at 05:39:25PM -0500, Jean Schurger wrote:
What's the 'good' way to ask to review an update of a package like this
one ? I should continue to dput it as replacement, and ask in that
thread ?
   
   dput -f mentors it will overwrite the previous package. You can
   state that you updated it in this thread and it will probably be fine.
  
  
  Hi, i've updated my free42 package.
  
  The package can be found on mentors.debian.net:
  - URL: http://mentors.debian.net/debian/pool/main/f/free42
  - Source repository: deb-src http://mentors.debian.net/debian unstable
  main contrib non-free
  - dget
  http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc
  
  Can you have a look ?
  
  Jean.
 
 Hello,
 
 Package builds cleanly and is lintian clean. That's good news :)
 
   - dpkg_shlibdeps does not complain anymore. It means that your patch
 works. However, you have included two patches, and one
 (0_less_libs.diff) does not apply (probably because your output
 directory was named 2). The second one is fine, though. You can
 delete the non-working one and remove it from
 debian/patches/series.

Oh yes, i did not see the second patch (I'm not sure how it appear,
maybe when i was playing with quilt).

   - you use a lot of calls to pwd in debian/rules. This is not
 necessary, relative paths work too : `pwd`/x is equivalent to x.
 Moreover, debhelper can help a lot here (dh $@).

I will clean that

   - Your manpages should describe how the binary is run. Does it have
 command-line options ? etc. That will be the first thing your
 users will try if they don't get how the package work. Other
 documentation should go… in the documentation :-) .

Well, there is only one option: -skin, I'll put it in the man page.
But the upstream comes with other documentation than two README files
that overlap. I will figure what to do with that. Might be an upstream
issue too.

   - Probably an upstream issue, but I find it confusing to ship two
 binaries that differ only in the way they interpret numbers. IMHO
 that should be a command-line switch or a menu option.

I've looked quickly to the sources, move that 'option' into a switch
will be a very big patch, but i think debian have a solution for that.
May be i can build two packages, and use update-alternatives to handle
that problem. Is that an acceptable solution for you ?

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-10 Thread Etienne Millon
Again, no need to CC me :)

 Oh yes, i did not see the second patch (I'm not sure how it appear,
 maybe when i was playing with quilt).

According to the patch header, it has been created by dpkg-source. I
can't remember the exact process, though.

 Well, there is only one option: -skin, I'll put it in the man page.
 But the upstream comes with other documentation than two README files
 that overlap. I will figure what to do with that. Might be an upstream
 issue too.

Documenting this and give a pointer to the full documentation may be
enough.
 
- Probably an upstream issue, but I find it confusing to ship two
  binaries that differ only in the way they interpret numbers. IMHO
  that should be a command-line switch or a menu option.
 
 I've looked quickly to the sources, move that 'option' into a switch
 will be a very big patch, but i think debian have a solution for that.
 May be i can build two packages, and use update-alternatives to handle
 that problem. Is that an acceptable solution for you ?

Alternatives makes sense when two packages provides the same service.
With free42, different users may use different settings (or, a user
may use different settings). Building two executables is fine. You can
even have a single manpage (see for example grep, egrep, rgrep).

Cheers

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-09 Thread Jean Schurger
Le jeudi 09 décembre 2010 à 08:59 +0100, Etienne Millon a écrit :
 On Wed, Dec 08, 2010 at 05:39:25PM -0500, Jean Schurger wrote:
  What's the 'good' way to ask to review an update of a package like this
  one ? I should continue to dput it as replacement, and ask in that
  thread ?
 
 dput -f mentors it will overwrite the previous package. You can
 state that you updated it in this thread and it will probably be fine.

Thanks.

Well, it's updated.

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-09 Thread Jean Schurger
Le jeudi 09 décembre 2010 à 08:59 +0100, Etienne Millon a écrit :
 On Wed, Dec 08, 2010 at 05:39:25PM -0500, Jean Schurger wrote:
  What's the 'good' way to ask to review an update of a package like this
  one ? I should continue to dput it as replacement, and ask in that
  thread ?
 
 dput -f mentors it will overwrite the previous package. You can
 state that you updated it in this thread and it will probably be fine.


Hi, i've updated my free42 package.

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/f/free42
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

Can you have a look ?

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


RFS: free42

2010-12-08 Thread Jean Schurger

Dear mentors,

I am looking for a sponsor for my package free42.

* Package name: free42
  Version : 1.4.66-1
  Upstream Author : Thomas Okken
* URL : http://thomasokken.com/free42/
* License : GPL2
  Section : misc

It builds these binary packages:
free42 - HP42S Emulator

The package appears to be lintian clean.

My motivation for maintaining this package is:
This software maintained by its author and really usefull,
and it's a free implementation.

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/f/free42
- Source repository: deb-src http://mentors.debian.net/debian unstable
main contrib non-free
- dget
http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

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

Kind regards
Jean Schurger
-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-08 Thread Etienne Millon
Hello,

I am not a DD and thus can not sponsor you. However I had a look at
your package.

  - It fails to build in a clean chroot. At the beginning of the build it
complains about missing headers :

touch symlinks
g++ -MMD -Wall -g -I/usr/X11R6/include -fno-exceptions -fno-rtti  
-DVERSION=\1.4.66\ -c -o shell_main.o shell_main.cc
shell_main.cc:20:21: warning: gtk/gtk.h: No such file or directory
shell_main.cc:21:28: warning: gdk/gdkkeysyms.h: No such file or 
directory
shell_main.cc:22:22: warning: gdk/gdkx.h: No such file or directory
In file included from shell_main.cc:35:
shell_main.h:25: error: expected initializer before '*' token
shell_main.h:42: warning: 'typedef' was ignored in this declaration

You probably need to adjust Build-Dependencies to include libgtk2-dev
instead ok libgtk2.

  - debian/rules: I believe it can be simplified if you use the dh
$@ technique. rm ChangeLog || true should be changed to rm -f
Changelog as the former ignores all errors, not only this file
does not exist.

  - debian/changelog should mention that it is a new package and that
this upload would close an ITP. The 3.0 (quilt) format is not
relevant here as it has not changed.

As I couldn't build the package, I can't help you more, but I
encourage you to use a build system such as cowbuilder, that isolates
the package from your host system.

Hope that helps,

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-08 Thread Benoît Knecht
Hi Jean,

Jean Schurger wrote:
 I am looking for a sponsor for my package free42.
 
 * Package name: free42
   Version : 1.4.66-1
   Upstream Author : Thomas Okken
 * URL : http://thomasokken.com/free42/
 * License : GPL2
   Section : misc
 
 It builds these binary packages:
 free42 - HP42S Emulator
 
 The package appears to be lintian clean.

After applying the fix suggested by Etienne, I managed to build your
package. I did not review it yet, but there are a few lintian warnings
you may want to have a look into:

  I: free42 source: binary-control-field-duplicates-source field section in 
package free42
  I: free42 source: debian-watch-file-is-missing
  I: free42: extended-description-is-probably-too-short
  P: free42: no-homepage-field
  W: free42: new-package-should-close-itp-bug
  W: free42: binary-without-manpage usr/bin/free42bin
  W: free42: binary-without-manpage usr/bin/free42dec
  I: free42: desktop-entry-contains-encoding-key 
/usr/share/applications/free42bin.desktop:2 Encoding
  W: free42: desktop-entry-invalid-category Categories=GTK 
/usr/share/applications/free42bin.desktop
  I: free42: desktop-entry-contains-encoding-key 
/usr/share/applications/free42dec.desktop:2 Encoding
  W: free42: desktop-entry-invalid-category Categories=GTK 
/usr/share/applications/free42dec.desktop

(You can get details about these messages by running
'lintian -iI --pedantic' on your .changes file.)

Cheers,

-- 
Benoît Knecht


-- 
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/20101208160600.gb10...@lin51.tphys.uni-heidelberg.de



Re: RFS: free42

2010-12-08 Thread Jean Schurger
Le mercredi 08 décembre 2010 à 17:06 +0100, Benoît Knecht a écrit :
 Hi Jean,
 
 Jean Schurger wrote:
  I am looking for a sponsor for my package free42.
  
  * Package name: free42
Version : 1.4.66-1
Upstream Author : Thomas Okken
  * URL : http://thomasokken.com/free42/
  * License : GPL2
Section : misc
  
  It builds these binary packages:
  free42 - HP42S Emulator
  
  The package appears to be lintian clean.
 
 After applying the fix suggested by Etienne, I managed to build your
 package. I did not review it yet, but there are a few lintian warnings
 you may want to have a look into:
 
   I: free42 source: binary-control-field-duplicates-source field section in 
 package free42
   I: free42 source: debian-watch-file-is-missing
   I: free42: extended-description-is-probably-too-short
   P: free42: no-homepage-field
   W: free42: new-package-should-close-itp-bug
   W: free42: binary-without-manpage usr/bin/free42bin
   W: free42: binary-without-manpage usr/bin/free42dec
   I: free42: desktop-entry-contains-encoding-key 
 /usr/share/applications/free42bin.desktop:2 Encoding
   W: free42: desktop-entry-invalid-category Categories=GTK 
 /usr/share/applications/free42bin.desktop
   I: free42: desktop-entry-contains-encoding-key 
 /usr/share/applications/free42dec.desktop:2 Encoding
   W: free42: desktop-entry-invalid-category Categories=GTK 
 /usr/share/applications/free42dec.desktop
 
 (You can get details about these messages by running
 'lintian -iI --pedantic' on your .changes file.)


I've updated my packages fixing the problems, can you have a
look ?

http://mentors.debian.net/debian/pool/main/f/free42/

Jean.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-08 Thread Etienne Millon
On Wed, Dec 08, 2010 at 03:30:36PM -0500, Jean Schurger wrote:
 I've updated my packages fixing the problems, can you have a
 look ?
 
 http://mentors.debian.net/debian/pool/main/f/free42/
 
 Jean.

Hello Jean,

I could build your package. Here are a few remarks :

  - lintian still complains about missing manpages for free42bin and
free42dec. Your users will probably do, too :-)

  - dpkg-shlibdeps seems to complain about useless dependencies on

  libfontconfig.so.1 
  libatk-1.0.so.0 
  librt.so.1 
  libgio-2.0.so.0 
  libcairo.so.2 
  libpango-1.0.so.0 
  libgmodule-2.0.so.0 
  libgthread-2.0.so.0 
  libpangocairo-1.0.so.0 
  libfreetype.so.6 
  libpangoft2-1.0.so.0 

Those come from your Makefile which calls pkg-config --libs
gtk+-2.0. I am not sure about the best solution for this one.
It's only a warning, though.

Cheers

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-08 Thread Jean Schurger
Le mercredi 08 décembre 2010 à 21:56 +0100, Etienne Millon a écrit :
 On Wed, Dec 08, 2010 at 03:30:36PM -0500, Jean Schurger wrote:
  I've updated my packages fixing the problems, can you have a
  look ?
  
  http://mentors.debian.net/debian/pool/main/f/free42/
  
  Jean.
 
 Hello Jean,
 
 I could build your package. Here are a few remarks :
 
   - lintian still complains about missing manpages for free42bin and
 free42dec. Your users will probably do, too :-)

Yes, i don't know what to do with that. I should not use the
'unodcumented' because i've understood that it is/will be deprecated.

And i have no manual for that on the software sources.
Is there a template of manual that i can use ?

   - dpkg-shlibdeps seems to complain about useless dependencies on
 
   libfontconfig.so.1 
   libatk-1.0.so.0 
   librt.so.1 
   libgio-2.0.so.0 
   libcairo.so.2 
   libpango-1.0.so.0 
   libgmodule-2.0.so.0 
   libgthread-2.0.so.0 
   libpangocairo-1.0.so.0 
   libfreetype.so.6 
   libpangoft2-1.0.so.0 
 
 Those come from your Makefile which calls pkg-config --libs
 gtk+-2.0. I am not sure about the best solution for this one.
 It's only a warning, though.
 

Yes, i was knowing that too, the Makefile is part of the sources,
should i patch it to prevent thoses links ? Free42 is linked
indirectly to those libraries as they are gtk+ dependencies, and
free42 use gtk+.

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-08 Thread Etienne Millon
(no need to CC me, I am subscribed to the mailing-list)

On Wed, Dec 08, 2010 at 04:04:09PM -0500, Jean Schurger wrote:
 Yes, i don't know what to do with that. I should not use the
 'unodcumented' because i've understood that it is/will be deprecated.
 
 And i have no manual for that on the software sources.
 Is there a template of manual that i can use ?

You can for example learn from an existing manpage (they are text
source files). There are plenty of them in /usr/share/man :-)

You can also use a compiler that will produce a manpage from a
(simpler) description. I've used pandoc (packaged in Debian) which
does the job.

Once it's done you should send the manpages to upstream, too.

- dpkg-shlibdeps seems to complain about useless dependencies on
  
libfontconfig.so.1 
libatk-1.0.so.0 
librt.so.1 
libgio-2.0.so.0 
libcairo.so.2 
libpango-1.0.so.0 
libgmodule-2.0.so.0 
libgthread-2.0.so.0 
libpangocairo-1.0.so.0 
libfreetype.so.6 
libpangoft2-1.0.so.0 
  
  Those come from your Makefile which calls pkg-config --libs
  gtk+-2.0. I am not sure about the best solution for this one.
  It's only a warning, though.
  
 
 Yes, i was knowing that too, the Makefile is part of the sources,
 should i patch it to prevent thoses links ? Free42 is linked
 indirectly to those libraries as they are gtk+ dependencies, and
 free42 use gtk+.

If there's actually a way to build in a cleaner way (and remove
explicit dependencies), you should patch the upstream sources. As
you're using the new 3.0 (quilt) format, it means recording a patch
and putting it in debian/patches. You can do that by hand (tedious),
or directly with quilt. If you are using a higher level system
(git-buildpackage, …), there should be a direct way to do that, too.
Upstream will probably be happy to merge this patch in their next
version, too (once again, assuming that it's not a false warning).

-- 
Etienne Millon


signature.asc
Description: Digital signature


Re: RFS: free42

2010-12-08 Thread Jean Schurger
Le mercredi 08 décembre 2010 à 22:21 +0100, Etienne Millon a écrit :
 (no need to CC me, I am subscribed to the mailing-list)
 
 On Wed, Dec 08, 2010 at 04:04:09PM -0500, Jean Schurger wrote:
  Yes, i don't know what to do with that. I should not use the
  'unodcumented' because i've understood that it is/will be deprecated.
  
  And i have no manual for that on the software sources.
  Is there a template of manual that i can use ?
 
 You can for example learn from an existing manpage (they are text
 source files). There are plenty of them in /usr/share/man :-)
 
 You can also use a compiler that will produce a manpage from a
 (simpler) description. I've used pandoc (packaged in Debian) which
 does the job.
 
 Once it's done you should send the manpages to upstream, too.
 
 - dpkg-shlibdeps seems to complain about useless dependencies on
   
 libfontconfig.so.1 
 libatk-1.0.so.0 
 librt.so.1 
 libgio-2.0.so.0 
 libcairo.so.2 
 libpango-1.0.so.0 
 libgmodule-2.0.so.0 
 libgthread-2.0.so.0 
 libpangocairo-1.0.so.0 
 libfreetype.so.6 
 libpangoft2-1.0.so.0 
   
   Those come from your Makefile which calls pkg-config --libs
   gtk+-2.0. I am not sure about the best solution for this one.
   It's only a warning, though.
   
  
  Yes, i was knowing that too, the Makefile is part of the sources,
  should i patch it to prevent thoses links ? Free42 is linked
  indirectly to those libraries as they are gtk+ dependencies, and
  free42 use gtk+.
 
 If there's actually a way to build in a cleaner way (and remove
 explicit dependencies), you should patch the upstream sources. As
 you're using the new 3.0 (quilt) format, it means recording a patch
 and putting it in debian/patches. You can do that by hand (tedious),
 or directly with quilt. If you are using a higher level system
 (git-buildpackage, …), there should be a direct way to do that, too.
 Upstream will probably be happy to merge this patch in their next
 version, too (once again, assuming that it's not a false warning).
 

Thanks, i'll fix thoses two issues.

What's the 'good' way to ask to review an update of a package like this
one ? I should continue to dput it as replacement, and ask in that
thread ?

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


RFS: free42

2010-12-08 Thread Jean Schurger
From: Jean Schurger j...@schurger.org
To: debian-mentors@lists.debian.org
Subject: RFS: free42

Dear mentors,

I am looking for a sponsor for my package free42.

* Package name: free42
  Version : 1.4.66-1
  Upstream Author : http://thomasokken.com/free42/
* URL : Thomas Okken
* License : GPL-2
  Section : misc

It builds these binary packages:
free42 - HP42S Emulator

The package appears to be lintian clean.

The upload would fix these bugs: 606358

My motivation for maintaining this package is: this nice piece of software 
should be available in debian.

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/f/free42
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget http://mentors.debian.net/debian/pool/main/f/free42/free42_1.4.66-1.dsc

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

Kind regards

-- 
Jean Schurger
http://schurger.org
GPG: http://schurger.org/jean.asc



signature.asc
Description: This is a digitally signed message part


Re: RFS: free42

2010-12-08 Thread Etienne Millon
On Wed, Dec 08, 2010 at 05:39:25PM -0500, Jean Schurger wrote:
 What's the 'good' way to ask to review an update of a package like this
 one ? I should continue to dput it as replacement, and ask in that
 thread ?

dput -f mentors it will overwrite the previous package. You can
state that you updated it in this thread and it will probably be fine.

-- 
Etienne Millon


-- 
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/20101209075908.ga11...@john.ssi.corp