Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Frederic Bonnard]

Hi Gianfranco, Paul, thank you for assisting me.
I put the binary in a separated package.
Concerning the symbol file, it should be much better (after some study,
I realize, it wasn't clean at all :) )
I use now dh --with pkgkde_symbolshelper and this enables to use a oneliner such
as :
(subst)_Z30SPHLockFreeHeapNearAllocNoLockPv{size_t}@Base 1.1.0

It adds one Build-Depends. I can make use of arch-bits, let me know what you
think is recommended.
Btw the (c++) tag would be nice to use with very readable function names, but it
seems it has no support for {size_t} and on the other side pkgkde_symbolshelper
seems not to have support for (c++), or I missed something.

Fred

On Mon, 18 Apr 2016 17:03:24 + (UTC), Gianfranco Costamagna 
 wrote:
> Hi,
> 
> 
> >> BTW one single symbol file please>I recommend using the C++ support in 
> >> dpkg-gensymbols to achieve this,
> >it is better than restricting particular symbols to particular arches
> >etc.
> 
> this has to be read as:
> 
> "one single symbol file that covers every architectures please"
> 
> anyway, please convert them in c++ symbols.
> 
> thanks
> 
> G.
> 

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Gianfranco Costamagna]

Hi,


>> BTW one single symbol file please>I recommend using the C++ support in 
>> dpkg-gensymbols to achieve this,
>it is better than restricting particular symbols to particular arches
>etc.

this has to be read as:

"one single symbol file that covers every architectures please"

anyway, please convert them in c++ symbols.

thanks

G.

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Paul Wise]

On Fri, Apr 15, 2016 at 8:59 PM, Gianfranco Costamagna wrote:

> BTW one single symbol file please

I recommend using the C++ support in dpkg-gensymbols to achieve this,
it is better than restricting particular symbols to particular arches
etc.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Gianfranco Costamagna]

Hi again

BTW one single symbol file please

no need to have an i386 with just two symbols differing from powerpc
https://sources.debian.net/src/libechonest/2.3.1-0.3/debian/libechonest2.3.symbols/you
 can exclude them with (powerpc) or similar keywords


or something like this
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=820297

cheers,

G.


Il Venerdì 15 Aprile 2016 14:53, Gianfranco Costamagna 
 ha scritto:
Hi Frederic, still another question:

debian/rules.old <-- you might want to drop it


a libsphde1 library that has a binary...

usually libraries are installed by the dependencies, and the binaries are split
in separate packages.

please try to provide a package with the binary and the manpage, with an 
eventual
dependency on the libsphde1 library.


BTW std-version is not 3.9.8


g.



Il Venerdì 8 Aprile 2016 10:36, Frederic Bonnard  ha 
scritto:
Hi Jakub,

> *.manpages are mostly useful for manual pages that weren't installed by 
> the upstream build system. (Perhaps the dh_installmanpages documentation 
> should be more clear about it...)
> 
> If the upstream build system did install them, you should use *.install 
> files instead, where you won't need the debian/tmp prefix.

That makes sense indeed now.. with the fact that dh_installman was able to pick
within $(CURDIR).
I migrated that man page to the related .install file and it's simpler.

> Keep in mind that flawfinder's signal/noise ratio is very close to 0...

Should I be less scared ? ;)

The updated package is here :
http://mentors.debian.net/debian/pool/main/s/sphde/sphde_1.1.0-1.dsc

Thank you Jakub :)

F.

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Gianfranco Costamagna]

Hi Frederic, still another question:

debian/rules.old <-- you might want to drop it


a libsphde1 library that has a binary...

usually libraries are installed by the dependencies, and the binaries are split
in separate packages.

please try to provide a package with the binary and the manpage, with an 
eventual
dependency on the libsphde1 library.


BTW std-version is not 3.9.8


g.


Il Venerdì 8 Aprile 2016 10:36, Frederic Bonnard  ha 
scritto:
Hi Jakub,

> *.manpages are mostly useful for manual pages that weren't installed by 
> the upstream build system. (Perhaps the dh_installmanpages documentation 
> should be more clear about it...)
> 
> If the upstream build system did install them, you should use *.install 
> files instead, where you won't need the debian/tmp prefix.

That makes sense indeed now.. with the fact that dh_installman was able to pick
within $(CURDIR).
I migrated that man page to the related .install file and it's simpler.

> Keep in mind that flawfinder's signal/noise ratio is very close to 0...

Should I be less scared ? ;)

The updated package is here :
http://mentors.debian.net/debian/pool/main/s/sphde/sphde_1.1.0-1.dsc

Thank you Jakub :)

F.

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Frederic Bonnard]

Hi Jakub,

> *.manpages are mostly useful for manual pages that weren't installed by 
> the upstream build system. (Perhaps the dh_installmanpages documentation 
> should be more clear about it...)
> 
> If the upstream build system did install them, you should use *.install 
> files instead, where you won't need the debian/tmp prefix.

That makes sense indeed now.. with the fact that dh_installman was able to pick
within $(CURDIR).
I migrated that man page to the related .install file and it's simpler.

> Keep in mind that flawfinder's signal/noise ratio is very close to 0...

Should I be less scared ? ;)

The updated package is here :
http://mentors.debian.net/debian/pool/main/s/sphde/sphde_1.1.0-1.dsc

Thank you Jakub :)

F.

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Jakub Wilk]

* Jakub Wilk , 2016-04-07, 14:55:
*.manpages are mostly useful for manual pages that weren't installed by 
the upstream build system. (Perhaps the dh_installmanpages 
documentation should be more clear about it...)


Oops, I meant s/dh_installmanpages/dh_installman/.

--
Jakub Wilk

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Jakub Wilk]

* Frederic Bonnard , 2016-04-07, 14:34:
debian/*.manpages, you can remove the "debian/tmp" prepending path I 
think

surprisingly, I failed to do without.


*.manpages are mostly useful for manual pages that weren't installed by 
the upstream build system. (Perhaps the dh_installmanpages documentation 
should be more clear about it...)


If the upstream build system did install them, you should use *.install 
files instead, where you won't need the debian/tmp prefix.



flawfinder -Q -c .
[lots]

aw.. there's a bunch of flaws :)


Keep in mind that flawfinder's signal/noise ratio is very close to 0...

--
Jakub Wilk

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Frederic Bonnard]

Hi Gianfranco,
thanks for your comments as usual,

> control: std-version is now 3.9.7

updated

> libjs-query can be left as-is, because using the internal debian jquery might 
> result in a bad documentation
> package (look e.g. 
> https://lists.debian.org/debian-devel/2014/10/msg00774.html;)
> this can simplify rules, control and so on

ok, I removed jquery related things (Depends in d/control and d/rules). Deep
issue actually.

> debian/*.manpages, you can remove the "debian/tmp" prepending path I think

surprisingly, I failed to do without. Maybe because dh_installman should be 
able to
take files in $CURDIR also ?

> copyright: is the debian directory under the same author as upstream?
> so you can avoid mentioning it.

Actually no. I put an Author field now, but I don't see that field in the
debian policy. Though I saw it used in several d/copyright browsing
sources.debian.net. Any tip ?

> package FTBFS on i386 and probably everywhere except amd64 (tested amd64, 
> arm64 and i386 for now).

I did a patch in the package (already merged upstream) for i386 (missing
bracket in the source ..). It compiled on amd64, i386, ppc64el, powerpc. I'll
try to setup an arm qemu VM to check more.

> check-all-the-things:
> flawfinder -Q -c .
> [lots]

aw.. there's a bunch of flaws :)
I'm going to sensitize upstream about that. 

> codespell --quiet-level=3
> [lots]

I did a patch in the package and also submitted it upstream.

> some stuff autogenerated seems to be GPL-2 (check-all-the-things output)

I added config.sub and config.guess into d/copyright as they were distributed
in the source package indeed.

F.

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Gianfranco Costamagna]

control: owner -1 !
control: tags -1 moreinfo

Hi,
control: std-version is now 3.9.7

libjs-query can be left as-is, because using the internal debian jquery might 
result in a bad documentation
package (look e.g. https://lists.debian.org/debian-devel/2014/10/msg00774.html;)

this can simplify rules, control and so on

debian/*.manpages, you can remove the "debian/tmp" prepending path I think

copyright: is the debian directory under the same author as upstream?
so you can avoid mentioning it.


package FTBFS on i386 and probably everywhere except amd64 (tested amd64, arm64 
and i386 for now).


check-all-the-things:
flawfinder -Q -c .

[lots]
codespell --quiet-level=3
[lots]
some stuff autogenerated seems to be GPL-2 (check-all-the-things output)

the other stuff LGTM.

cheers,

G.



Il Martedì 16 Febbraio 2016 17:03, Frederic Bonnard  
ha scritto:
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "sphde"

Package name: sphde
Version : 1.1.0-1
Upstream Author : SPHDE team
URL : https://github.com/sphde/sphde
License : EPL-1
Section : devel

It builds those binary packages:

  libsphde-dev - Shared Persistent Heap Data Environment library development 
files
  libsphde-doc - Shared Persistent Heap Data Environment library documentation 
files
  libsphde1  - Shared Persistent Heap Data Environment library

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

  https://mentors.debian.net/package/sphde


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

dget -x http://mentors.debian.net/debian/pool/main/s/sphde/sphde_1.1.0-1.dsc

More information about sphde can be obtained from https://github.com/sphde/sphde

Regards,
Frederic Bonnard

Bug#814921: RFS: sphde/1.1.0-1 [ITP] -- sphde -- Shared Persistent Heap Data Environment library

[Frederic Bonnard]

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "sphde"

 Package name: sphde
 Version : 1.1.0-1
 Upstream Author : SPHDE team
 URL : https://github.com/sphde/sphde
 License : EPL-1
 Section : devel

It builds those binary packages:

  libsphde-dev - Shared Persistent Heap Data Environment library development 
files
  libsphde-doc - Shared Persistent Heap Data Environment library documentation 
files
  libsphde1  - Shared Persistent Heap Data Environment library

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

  https://mentors.debian.net/package/sphde


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

dget -x http://mentors.debian.net/debian/pool/main/s/sphde/sphde_1.1.0-1.dsc

More information about sphde can be obtained from https://github.com/sphde/sphde

Regards,
 Frederic Bonnard