Bug#835274: dh-text no longer needed

2016-11-25 Thread Gianfranco Costamagna
>Seems you used git. I did forgot to push, sorry. Now last version is published

>both on mentors and in git
>(https://anonscm.debian.org/cgit/users/kaction-guest/bcron.git)


I used also mentors, and it is still outdated>Nobody should use dh-text. With 
recent dpkg-dev, which provide S:foo
>substitution, dh-text should be obliterated.


ok, but the package FTBFS
http://debomatic-amd64.debian.net/distribution#unstable/bcron/0.10-4/buildlog

G.



Bug#835274: dh-text no longer needed

2016-11-23 Thread Gianfranco Costamagna
and btw, I don't like this dh_text new feature :/
bcron - ${text:ShortDesc} (programs)
bcron-run - ${text:ShortDesc} (integration with runit)

this is shown in the web page...

probably this will break PTS too

somebody should probably try to fix mentors and PTS before start using this new 
key

G.



Bug#835274: dh-text no longer needed

2016-11-23 Thread Gianfranco Costamagna


>Done. Updated version on mentors.



that version still build-depends on dh_text

G.



Bug#835274: dh-text no longer needed

2016-11-18 Thread Dmitry Bogatov

[2016-11-18 08:54] Christian Seiler 
> I'm hence removing the moreinfo tag, because your course of action
> to remove dietlibc-dev from Build-Depends of your package is the
> right one. If at some point in the future bglibs are recompiled
> against dietlibc (in addition to glibc preferably) it might make
> sense to re-add the dietlibc-dev Build-Depends here, but for now
> you can't do anything about that.
>
> I would only ask you to alter the changelog to reflect the actual
> reason why you don't use dietlibc anymore; that is use an entry
> like:
>
>  * Drop dietlibc-dev from Build-Depends for now. (Libraries
>depended upon are not compiled for dietlibc anymore.)

Done. Updated version on mentors.

-- 
X-Web-Site: https://sinsekvu.github.io | Note that I process my email in batch,
Accept-Languages: eo,ru,en | at most once every 24 hours. If matter
Accept: text/plain, text/x-diff| is urgent, you have my phone number.


pgpBrKwwXVlVa.pgp
Description: PGP signature


Bug#835274: dh-text no longer needed

2016-11-17 Thread Christian Seiler
Control: tags -1 - moreinfo

On 11/18/2016 08:34 AM, Dmitry Bogatov wrote:
> 
> [2016-11-16 13:09] Christian Seiler 
>> Am 16. November 2016 10:28:41 MEZ, schrieb Dmitry Bogatov :
>>>  * Drop diet libc build due issues with errno
>>
>> As a current co-maintainer of dietlibc in Debian, could you elaborate
>> here? I've spent the last couple of months fixing all sorts of bugs in
>> there (and impro ving packaging, for example dietlibc-dev is now M-A:
>> same), and if problems remain, I'd also like to see them fixed.
>>
>> Of course, you are free to use/drop usage of dietlibc for whatever
>> reason, and maybe there are others than just a specific bug. But
>> irrespective of whether y ou keep building against it, I'd like to fix
>> potential bugs.
> 
> Hm, seems things changed since last time I tried to build with diet. But
> still, it does not work:
> 
>   ./load bcrontab bcron.a -lbg-cli -lbg
>   ./load bcron-start bcron.a -lbg
>   /usr/lib/bglibs/libbg.a(connectu.o): In function `socket_connectu':
>   (.text+0x2f): undefined reference to `__strcpy_chk'
>   /usr/lib/bglibs/libbg.a(mktemp.o): In function `path_mktemp':
>   (.text+0xd7): undefined reference to `__lxstat'
>   collect2: error: ld returned 1 exit status
>   Makefile:66: recipe for target 'bcrontab' failed

Ah, you're trying to link code compiled against glibc headers (in
this case bglibs) with dietlibc, and that won't work. If you want
to use dietlibc in conjunction with additional libraries, all
additional libraries need to be compiled for dietlibc as well.

You can easily do that additionally, by compiling the libraries
twice: installing the libraries in /usr/lib/ directly
when compiling against glibc, and installing them in
$(diet -L ${CC:-gcc)) [1] when compiling against dietlibc.

Looking at the changelog of bglibs, they haven't compiled against
dietlibc since 2008, so unless you change bglibs, this can't
work.

I'm hence removing the moreinfo tag, because your course of action
to remove dietlibc-dev from Build-Depends of your package is the
right one. If at some point in the future bglibs are recompiled
against dietlibc (in addition to glibc preferably) it might make
sense to re-add the dietlibc-dev Build-Depends here, but for now
you can't do anything about that.

I would only ask you to alter the changelog to reflect the actual
reason why you don't use dietlibc anymore; that is use an entry
like:

 * Drop dietlibc-dev from Build-Depends for now. (Libraries
   depended upon are not compiled for dietlibc anymore.)

That makes it clear what is really going on.

Thanks!

Regards,
Christian

[1] This is Multi-Arch-aware in the current Debian package for
dietlibc. For example, on x86_64 it will expand to
/usr/lib/x86_64-linux-gnu/diet/lib-x86_64
Headers need to go into /usr/include/diet, and need to be
the same across all architectures if the package containing
them is to be M-A: same. I'll probably need to change that
part in the future.



Bug#835274: dh-text no longer needed

2016-11-17 Thread Dmitry Bogatov

[2016-11-16 13:09] Christian Seiler 
> Am 16. November 2016 10:28:41 MEZ, schrieb Dmitry Bogatov :
> >  * Drop diet libc build due issues with errno
>
> As a current co-maintainer of dietlibc in Debian, could you elaborate
> here? I've spent the last couple of months fixing all sorts of bugs in
> there (and impro ving packaging, for example dietlibc-dev is now M-A:
> same), and if problems remain, I'd also like to see them fixed.
>
> Of course, you are free to use/drop usage of dietlibc for whatever
> reason, and maybe there are others than just a specific bug. But
> irrespective of whether y ou keep building against it, I'd like to fix
> potential bugs.

Hm, seems things changed since last time I tried to build with diet. But
still, it does not work:

./load bcrontab bcron.a -lbg-cli -lbg
./load bcron-start bcron.a -lbg
/usr/lib/bglibs/libbg.a(connectu.o): In function `socket_connectu':
(.text+0x2f): undefined reference to `__strcpy_chk'
/usr/lib/bglibs/libbg.a(mktemp.o): In function `path_mktemp':
(.text+0xd7): undefined reference to `__lxstat'
collect2: error: ld returned 1 exit status
Makefile:66: recipe for target 'bcrontab' failed

Seems it is somewhat issue with bglibs, which also need some love. But I
would like to not steal one more package from Gerrit Pape. At least
not until everything runit-related is in perfect shape.

But every Debian package, build-able with diet libc makes me little happier.

-- 
X-Web-Site: https://sinsekvu.github.io | Note that I process my email in batch,
Accept-Languages: eo,ru,en | at most once every 24 hours. If matter
Accept: text/plain, text/x-diff| is urgent, you have my phone number.


pgpV19jcJsd8K.pgp
Description: PGP signature


Bug#835274: dh-text no longer needed

2016-11-16 Thread Gianfranco Costamagna
control: tags -1 moreinfo


>Of course, you are free to use/drop usage of dietlibc for whatever reason, and 
>maybe there are others than just a specific bug. But irrespective of whether 
>you keep >building against it, I'd like to fix potential bugs.



this is worth a moreinfo tag :)


G.



Bug#835274: dh-text no longer needed

2016-11-16 Thread Christian Seiler
Hi there,

Am 16. November 2016 10:28:41 MEZ, schrieb Dmitry Bogatov :
>  * Drop diet libc build due issues with errno

As a current co-maintainer of dietlibc in Debian, could you elaborate here? 
I've spent the last couple of months fixing all sorts of bugs in there (and 
improving packaging, for example dietlibc-dev is now M-A: same), and if 
problems remain, I'd also like to see them fixed.

Of course, you are free to use/drop usage of dietlibc for whatever reason, and 
maybe there are others than just a specific bug. But irrespective of whether 
you keep building against it, I'd like to fix potential bugs.

Thanks,
Christian



Bug#835274: dh-text no longer needed

2016-11-16 Thread Dmitry Bogatov

control: tags -1 -moreinfo
control: unblock -1 by 834313
control: close 834313

With dpkg-dev >= 1.18.11, implementing S:field substitution, dh-text
is no longer needed, this request is active again.

In attachment there is an email from current maintainer (in CC), giving
me permission to make these changes on his behalf.

Changes since last upload:

  * Write debian/watch
  * Change source package format to quilt, remove manual patches
application
  * Convert package to use debhelper
  * Use `dh-runit' to install runit scripts and generate log scripts
(Closes: #832656)
  * Use `dh-buildinfo' to simplify tracking bugs, related to build-tools
  * Avoid need of hand-written maintainer scripts by
use of `dh-runit' and `dh-sysuser'
  * Drop diet libc build due issues with errno
  * Remove no longer needed README files
  * Move 'debian/crontab' into 'debian/contrib' for more clean package layout
  * Enable hardening
  * Fix @dircategory in texinfo manual
  * Install `doc-base' document
  * Add Homepage field
  * Bump standards version to 3.9.8 (no changes needed)
  * Remove Section field from `bcron-run' field, since it duplicated one
defined in first paragraph.
  * Update Vcs-Git and Vcs-Browser fields
  * Convert `debian/copyrigh' to dep5 format
  * Use S:fieldname substitution to avoid duplication in `debian/control'.
+ Add versioned dependency on (dpkg-dev >= 1.18.11)

Regards,
  Dmitry Bogatov

-- 
X-Web-Site: https://sinsekvu.github.io | Note that I process my email in batch,
Accept-Languages: eo,ru,en | at most once every 24 hours. If matter
Accept: text/plain, text/x-diff| is urgent, you have my phone number.
--- Begin Message ---
On Sat, Jul 30, 2016 at 01:53:50PM +0300, Dmitry Bogatov wrote:
> I investigated issue a bit more. From all listed -run packages, currently
> only following are problematic:
> 
>   bcron-run
>   twoftpd-run
>   tinydns-run
> 
> I started fixing bcron-run, and discovered, that to use dh_runit, I
> need also at least dh_installdeb, dh_prep and dh_gencontrol. And this is
> on half way to converting to debhelper (which is inapporiate for NMU).
> 
> Here I ask you for permission:
> 
>  - convert those 3 packages to debhelper and upload them with myself
>as maintainer (unfortunately, I can't do this in NMU). Feel free
>to add 'I reserve right to take this package back' clause to your
>permission.
> 
>  - the same for rest of -run packages. This is not so urgent, since
>right now they are coinstallable with runit/unstable, and I do not
>plan to touch them, as long as they just work.
> 
> I am sorry about all this situation, but since I need to resolve
> (Bug#832656), I am forced to bother you. And since this bug is
> serious, I really want to resolve it till freeze. Dropping a mail
> on debian-mentors should suffice as permission.

Hi Dmitry,

I'm still much interested in the djbdns (provides tinydns-run) and bcron
packages, while twoftpd isn't that fency any more.

If you want, please take over maintainership for twoftpd.  For the other
two packages, I suggest you switch them to debhelper (I never liked
that) with my permission, since it appears to be really necessary.  Add
yourself as uploader, but please (not yet) switch the maintainer.

Best Regards, thanks for all the time you put into Debian, Gerrit.

--- End Message ---