Help with build issue

[Hugo Santos]

Hi guys,

I'm the developer (and maintainer) of the debian package "mrd6".
Recently i had a small issue with building on S390 (issues with
handling size_t). Unfortunately due to time constraints this went
unfixed for some months but i would like to finally close this problem
(bug #466838). I was wondering if someone with access to a S390 system
could try and build the package (on etch or newer) to check if the
issue is fixed. I'm asking so i don't make the upload a trial and
error procedure. :-)

You can find the tentative version to be upload at:
http://fivebits.net/~hugo/mrd6-build/

Thanks in advance,
Hugo

Hi Lior,
Like i mentioned before this should be fixed in the upstream source.
However i've prepared the debian package to be uploaded, you can fetch
http://fivebits.net/~hugo/mrd6-build/
As i said in the previous email, fixing this issue was non-trivial
due to particularities of S390, size_t there is different from
everywhere else. Not having a S390 system available to test the
compilation with reduces it very much into a trial and error procedure
i'm afraid.
Anyway, let met know if there's anything else i can do.
Thanks,
Hugo

Hi Hugo,
Any news with this bug ?
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=466838
--
Lior Kaplan
C644 D0B3 92F4 8FE4 4662 B541 1558 9445 99E8 1DA0

[Frans Pop]

Post by Hugo Santos
I'm the developer (and maintainer) of the debian package "mrd6".
Recently i had a small issue with building on S390 (issues with
handling size_t).

Package seems to build fine. I do get one warning, which should be trivial
to fix:
C++ timers.cpp
timers.cpp: In member function ‘bool timermgr::output_info(base_stream&, bool) const’:
timers.cpp:406: warning: format ‘%i’ expects type ‘int’, but argument 4 has type ‘size_t’

I noticed that your package is packaged in "Debian native" format, which
does not seem appropriate. Suggest you use an .orig tarball even if you
are both upstream and Debian maintainer.

Cheers,
FJP

[Hugo Santos]

Hi Frans,

Post by Frans Pop
Package seems to build fine. I do get one warning, which should be trivial
C++ timers.cpp
timers.cpp:406: warning: format '%i' expects type 'int', but argument 4 has type 'size_t'

Great! Thanks. I'll fix the warning as well.

Post by Frans Pop
I noticed that your package is packaged in "Debian native" format, which
does not seem appropriate. Suggest you use an .orig tarball even if you
are both upstream and Debian maintainer.

So is it ok to have a .orig tarball and no .diff? Since i keep the
debian/ files in the same repository whenever i package no patch is
required.

Thanks again,
Hugo

[Frans Pop]

Post by Hugo Santos

Post by Frans Pop
I noticed that your package is packaged in "Debian native" format,
which does not seem appropriate. Suggest you use an .orig tarball
even if you are both upstream and Debian maintainer.

So is it ok to have a .orig tarball and no .diff? Since i keep the
debian/ files in the same repository whenever i package no patch is
required.

I don't really know. You'd better check Policy and the Developer Reference
on that and maybe ask on debian-devel.