<div dir="ltr"><div dir="ltr">On Tue, Dec 22, 2020 at 8:53 PM James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Summary: you are conflating git patches with diff patches.<br></blockquote><div><br></div><div>Agreed, makes sense now.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On Tue, Dec 22, 2020 at 12:55:49PM +0100, Chihurumnaya Ibiam wrote:<br>
> telepathy-salut-0.8.1.tar.gz does contain lib/ext/wocky as a<br>
> directory with files in it and it has the github repo as it's<br>
> upstream and that doesn't contain lib/ext /wocky as a directory but<br>
> as a wocky submodule. I don't see a reason why the git submodule is<br>
> affecting it at the moment<br>
<br>
It has affected it because of how you prepared the patch using git<br>
instead of using diff.<br></blockquote><div><br></div><div>Yeah I normally prepare such patches with git as most of the time the upstream on github is still active.</div><div>Now I know different.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Yeah I haven't created an PRs as I'm trying to get it to build<br>
> locally first, you can find the patch here.<br>
<br>
The patch is in error because;<br>
<br>
(a) it is generated using git(1), and contains no content except<br>
submodule hashes, yet you are asking to apply using patch(1),<br>
<br>
(b) unlike patch0, it does not have complete file paths.<br>
<br>
You should generate the patch again, using diff -ru, between two<br>
clones that have been prepared to the corresponding states.<br></blockquote><div><br></div><div>Yeah I will.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> The commit upstream shows that there was a change but the change was<br>
> to a submodule, the source files from <a href="http://telepathyfreedesktop.org" rel="noreferrer" target="_blank">telepathyfreedesktop.org</a><br>
> contains lib/ext/wocky but the source on github doesn't and that<br>
> explains the error from build.log<br>
<br>
You should not use the tar.gz from github, because it<br>
<br>
(a) does not equal the distributed software; therefore unsupported,<br>
<br>
(b) does not contain autoconf prepared source; therefore broken, and<br>
<br>
(c) does not have submodules expanded; therefore incomplete.<br>
<br>
> I used rpmbuild to build just the %prep section and ls shows there<br>
> files in lib /ext/wocky but still throws same error when applying<br>
> the patch and I don't get why right now but I think it's because the<br>
> patch is pointing to a submodule commit.<br>
<br>
No, it is because the patch is not intended for processing by<br>
patch(1).<br></blockquote><div><br></div><div>Yeah, makes sense now.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The patch(1) command has no clue (or glue) for handling git patches in<br>
that way.<br></blockquote><div><br></div><div>I understand now.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Hope that helps!<br></blockquote><div><br></div><div>Yeah, thanks a lot. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
James Cameron<br>
<a href="https://quozl.linux.org.au/" rel="noreferrer" target="_blank">https://quozl.linux.org.au/</a></blockquote><div><br></div><div><br></div><div><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><pre style="color:rgb(46,52,54);letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px"><span style="font-family:monospace,monospace">-- <br></span></pre><div style="color:rgb(46,52,54);font-size:14.6667px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px;width:71ch"><span style="font-family:monospace,monospace"><span></span><span></span>Ibiam Chihurumnaya <br></span></div><div style="color:rgb(46,52,54);font-size:14.6667px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px;width:71ch"><span style="font-family:monospace,monospace"><a href="mailto:ibiamchihurumnaya@gmail.com" style="color:rgb(42,118,198)" target="_blank">ibiamchihurumnaya@gmail.com</a></span></div><div style="color:rgb(46,52,54);font-size:14.6667px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px;width:71ch"><span style="font-family:monospace,monospace"><br><br></span></div></div></div></div></div></div> </div></div></div>