Re: RFS: kst (updated package)

2011-06-28 Thread Kilian Krause
Ruben,

On Tue, 2011-06-28 at 19:55 -0500, Ruben Molina wrote:
> El mar, 28-06-2011 a las 16:12 +0200, David Kalnischkies escribió:
> > On Tue, Jun 28, 2011 at 15:54, Ruben Molina  wrote:
> > > Thanks for your help!  Looks like it was wrong to assume a common set of
> > > symbols for all the arches.  May I ask you to remove debian/*.symbols,
> > > and try to build the package again?
> > 
> > Set in your debian/rules this instead of removal:
> > export DPKG_GENSYMBOLS_CHECK_LEVEL=0
> > 
> > It will cause dpkg-gensymbols to proceed regardless of the symbol-outcome
> 
> Thanks David,
> I re-uploaded the package following your advise.
> - dget http://mentors.debian.net/debian/pool/main/k/kst/kst_2.0.3-1.dsc

thanks. Comments are:

1. Still uses debhelper 7. Could/should be 8.

2. Your symbols obviously contain the debian version for
libkst2math2, libkst2core2, libkst2widgets2

You may want to try pkg-kde-tools using
(http://pkg-kde.alioth.debian.org/symbolfiles.html) in case your libs
are C++ to lessen the pain a bit.

3. You're not building a lib*-dbg package to help with debugging
including symbols. Would be nice to have one.

4. Just to make sure you've checked. There is a kst binary in stable.
Have you tested the upgrade path if this version enters testing and thus
will become the next stable version? The wnpp bug report reads that KST1
and KST2 are not fully feature-compatible.

5. You've shortened the description compared to the 1.7 version. This
may be a good thing, yet you've dropped entirely the remark that this is
a KDE app. Is it now GNOME/KDE/XFCE/... i.e. freedesktop.org compatible?

Anyhow. Build, signed, uploaded.

Thanks for stepping up as new maintainer.

-- 
Best regards,
Kilian





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


Re: RFS: kst (updated package)

2011-06-28 Thread Ruben Molina
El mar, 28-06-2011 a las 16:12 +0200, David Kalnischkies escribió:
> On Tue, Jun 28, 2011 at 15:54, Ruben Molina  wrote:
> > Thanks for your help!  Looks like it was wrong to assume a common set of
> > symbols for all the arches.  May I ask you to remove debian/*.symbols,
> > and try to build the package again?
> 
> Set in your debian/rules this instead of removal:
> export DPKG_GENSYMBOLS_CHECK_LEVEL=0
> 
> It will cause dpkg-gensymbols to proceed regardless of the symbol-outcome

Thanks David,
I re-uploaded the package following your advise.
- dget http://mentors.debian.net/debian/pool/main/k/kst/kst_2.0.3-1.dsc

Regards,
  Ruben


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


Re: RFS: kst (updated package)

2011-06-28 Thread Sune Vuorela
On 2011-06-28, David Kalnischkies  wrote:
> btw: Without looking at the source: Architecture dependent symbols are e.g.
> all methods with size_t (or its sibling ssize_t) or va_list as parameter.

and qreal if kst is what I think it is.

/Sune


-- 
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/slrnj0jq3d.p7v.nos...@sshway.ssh.pusling.com



Re: RFS: kst (updated package)

2011-06-28 Thread David Kalnischkies
On Tue, Jun 28, 2011 at 15:54, Ruben Molina  wrote:
> Thanks for your help!  Looks like it was wrong to assume a common set of
> symbols for all the arches.  May I ask you to remove debian/*.symbols,
> and try to build the package again?

Set in your debian/rules this instead of removal:

export DPKG_GENSYMBOLS_CHECK_LEVEL=0

It will cause dpkg-gensymbols to proceed regardless of the symbol-outcome
which is quiet handy to collect architecture dependent symbols in the
first upload and to see other differences based on toolchain (e.g. c++ lib
tend to generate different optional symbols with different compiler versions).

btw: Without looking at the source: Architecture dependent symbols are e.g.
all methods with size_t (or its sibling ssize_t) or va_list as parameter.


Best regards

David Kalnischkies


--
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/banlktinmeerm3ezuzgmynrbv+a0xwz-...@mail.gmail.com



Re: RFS: kst (updated package)

2011-06-28 Thread Ruben Molina
Hi Kilian,

El mar, 28-06-2011 a las 08:02 +0200, Kilian Krause escribió:
> On Mon, 2011-06-27 at 18:45 -0500, Ruben Molina wrote:
> > Dear mentors,
> > 
> > I am looking for a sponsor for the new version 2.0.3-1
> > of my package "kst".
> > 
> > The package can be found on mentors.debian.net:
> > - URL: http://mentors.debian.net/debian/pool/main/k/kst
> 
> sorry, but fails to build for me:
> http://people.debian.org/~kilian/kst_2.0.3-1_amd64.build
> 

Thanks for your help!  Looks like it was wrong to assume a common set of
symbols for all the arches.  May I ask you to remove debian/*.symbols,
and try to build the package again? 

Regards,
 Ruben


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


Re: RFS: kst (updated package)

2011-06-27 Thread Kilian Krause
Ruben,

On Mon, 2011-06-27 at 18:45 -0500, Ruben Molina wrote:
> Dear mentors,
> 
> I am looking for a sponsor for the new version 2.0.3-1
> of my package "kst".
> 
> It builds these binary packages:
> kst- scientific data plotting tool
> kst-data   - set of data files for kst
> kst-doc- set of tutorials for kst
> libkst2core2 - kst core library
> libkst2math2 - kst math library
> libkst2widgets2 - kst widgets library
> 
> The package appears to be lintian clean.
> 
> The upload would fix these bugs: 553793, 589685, 604335, 615738
> 
> The package can be found on mentors.debian.net:
> - URL: http://mentors.debian.net/debian/pool/main/k/kst
> - Source repository: deb-src http://mentors.debian.net/debian unstable main 
> contrib non-free
> - dget http://mentors.debian.net/debian/pool/main/k/kst/kst_2.0.3-1.dsc

sorry, but fails to build for me:
http://people.debian.org/~kilian/kst_2.0.3-1_amd64.build

-- 
Regards,
Kilian


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


RFS: kst (updated package)

2011-06-27 Thread Ruben Molina
Dear mentors,

I am looking for a sponsor for the new version 2.0.3-1
of my package "kst".

It builds these binary packages:
kst- scientific data plotting tool
kst-data   - set of data files for kst
kst-doc- set of tutorials for kst
libkst2core2 - kst core library
libkst2math2 - kst math library
libkst2widgets2 - kst widgets library

The package appears to be lintian clean.

The upload would fix these bugs: 553793, 589685, 604335, 615738

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/k/kst
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget http://mentors.debian.net/debian/pool/main/k/kst/kst_2.0.3-1.dsc

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

Kind regards
 Ruben Molina


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