<html><head></head><body bgcolor="#ffffff" text="#2e3436" link="#2a76c6" vlink="#2e3436"><div>Hi Paul,</div><div><br></div><div>On Wed, 2017-11-15 at 21:01 +0100, Paul Gevers wrote:</div><blockquote type="cite"><pre>I am perfectly fine with you experimenting in experimental.

I have a question regarding the patch, as you change generated *.inc
files, can't you generate those in the rules file instead of in the
patch?</pre></blockquote><div>That's a very good question and I've asked myself why not regenerate the .inc file? The issue is that if we do then we end with a possible source change that is not carried in a patch and this does not please to dpkg-source (or some other dpkg-* tools I don't remember well anymore). So either we strip this file from original tar and force regenerating it or we patch it. I'm open to both and know how to regenerate this file.</div><div><br></div><blockquote type="cite"><pre> My experience is that if upstream isn't going to carry the patch
it is hard to update the patch for files like for that every new
upstream release.</pre></blockquote><div>Yes this is obvious, but updating the patch can be as simple as: apply the patch to real source file and then regenerate inc file and then update the patch (kind of quilt commit)</div><div><br></div><blockquote type="cite"><pre>Also, I am not sure if we aren't already rebuilding
all generated *.inc files in our current build process. I have stripped
major blocks for our patches in the past, because they were totally
unneeded as the file was (or could be) regenerated.
</pre></blockquote><div>We are rebuilding some of them.</div><div>I think many of them are automatically regenerated by upstream make files.</div><blockquote type="cite"><pre>
Paul

PS: I would have appreciated it when you would have committed this on a
separate (experimental) branch.<br></pre><pre></pre></blockquote><div>Sorry I didn't think about it. we can cherry-pick to experimental and then push -f to master~, but seems our git config refuse forced update.</div><div>Do you have a clue other than git revert? otherwise it is also fine.</div><div><div style="white-space: normal;"><span><pre><pre>-- <br></pre>Cheers,
Abou Al Montacir</pre></span></div></div></body></html>