trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: July 2012

Re: [trinity-devel] TDE packaging for Debian, tqt3 and g++ 4.7.1

From: "Timothy Pearson" <kb9vqf@...>
Date: Fri, 27 Jul 2012 00:43:01 -0500
> On July 26, 2012 18:30:20 Timothy Pearson wrote:
>> <snip>
>>
>> > While looking through the build.log for tqt3 I kept getting distracted
>> by
>> > the
>> > piles of "suggest parentheses around ‘&&’ within ‘||’
>> > [-Wparentheses]"
>> > warnings--so I  went through the source and added in all the missing
>> > parentheses g++ was complaining about (plus fixed a few more warnings
>> > where
>> > the fix was just as obvious). The build log is much easier to read
>> now,
>> > and
>> > potentially problematic warnings (such as
>> [-Wdelete-non-virtual-dtor][1])
>> > are
>> > no longer buried among the noise.
>>
>> Great!  I wanted to do something similar but never found the time.
>>
>> > What is the best way for me to get the tqt3 source changes to you?
>>
>> Just send a patch to this list; I'll watch for it and update qt3/tqt3.
>
> So, just a simple:
>
> $ diff -ru {tde,build}/main/dependencies/tqt3 > tqt3.diff
>
> is all you want?

No, Debian applies patches that are not suitable for the generic source
tree when it builds a package.  You would first need to reverse out all of
the Debian specific patches, then run the diff command.

Tim