[Debian-med-packaging] Bug#749136: jellyfish: FTBFS with clang instead of gcc

[Martin Steghöfer]

Hi Andreas!


El 08/10/14 a les 21:06, Andreas Tille ha escrit:
> Would you volunteer to check the current Git repository regarding the
> clang issue?  It's not ready yet but the state is good enough to look
> into the clang build.

The current state of the package in Debian-Med's git repository has the 
exact same issues as upstream HEAD (no more, no less). Therefore it 
would be adequate to use the patching from my upstream pull request:

https://github.com/gmarcais/Jellyfish/pull/21

Btw. I think you're missing the tag "upstream/2.1.4" in order to make git-buildpackage build it without complaints.


>> Sorry I can't be of any help there. I have no clue about ruby packaging. 
> Me neither but it's done and thanks to super fast ftpmaster yaggo is
> just in unstable.

Cool, that was quick! :-)

>> For the record: The patch has been merged into upstream git
>> (https://github.com/gmarcais/Jellyfish/pull/21) and will therefore
>> be part of the next Release v2.1.5 - which could take a while to be
>> published.
> Since I'm not that deep into Github:  What would be the most
> straightforward way to obtain these patches (and turn them into
> quilt patches)?

You can just append a suffix ".patch" to the URLs of commits (e.g. 
https://github.com/martin-steghoefer/Jellyfish/commit/780f2ec4.patch) or 
complete pull requests (e.g. 
https://github.com/gmarcais/Jellyfish/pull/21.patch), which will give 
you a patch in git format. This is something that quilt already accepts, 
although it is probably not "debianized" enough to put it into a package 
like this (additional git-specific meta-data, wrong tagging format, 
patch divided in several sections by commits). You can fix that by 
letting quilt rewrite the patch with "dquilt refresh" (when it is the 
top of the quilt stack) and then adding the Debian tagging.

I've done it for the pull request that is relevant here and attached it 
to this message.

Cheers,
Martin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-compilation.patch
Type: text/x-diff
Size: 2647 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/debian-med-packaging/attachments/20141008/9563b9f3/attachment.patch>