Help with build issue

[Hugo Santos-4]

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

On Fri, Jul 4, 2008 at 3:41 PM, Hugo Santos <hugo@...> wrote:

> 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
> it from:
>
>  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
>
> On Sat, Jun 28, 2008 at 11:09 AM, Lior Kaplan <kaplan@...> wrote:
>> Hi Hugo,
>>
>> Any news with this bug ?
>>
>> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=466838
>>
>>
>> --
>> Lior Kaplan
>> kaplan@...
>>
>> GPG fingerprint:
>> C644 D0B3 92F4 8FE4 4662  B541 1558 9445 99E8 1DA0
>>
>
>
>

--
To UNSUBSCRIBE, email to debian-s390-REQUEST@...
with a subject of "unsubscribe". Trouble? Contact listmaster@...

[Frans Pop-3]

Hugo Santos wrote:
>   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-4]

Hi Frans,

On Mon, Jul 7, 2008 at 4:24 PM, Frans Pop <elendil@...> wrote:
> 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'

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

> 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


--
To UNSUBSCRIBE, email to debian-s390-REQUEST@...
with a subject of "unsubscribe". Trouble? Contact listmaster@...

[Frans Pop-3]

On Monday 07 July 2008, Hugo Santos wrote:
> > 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.