<div dir="ltr">Hi Yuri,<div><br></div><div>Sorry to be a thorn in the flesh in this one.</div><div><br></div><div>Which source code were you applying this patch against?</div><div><br></div><div>I applied this patch on the released squid-4.0.8.tar.xz and it doesn't apply cleanly for starters... Maybe I am doing it wrongly? Just doing patch < /path/to/patch</div><div><br></div><div><div>wash@mail:~/ILI/Squid/4.x/squid-4.0.8$ patch < ../squid-4-14633.patch</div><div>Hmm...  Looks like a unified diff to me...</div><div>The text leading up to this was:</div><div>--------------------------</div><div>|------------------------------------------------------------</div><div>|revno: 14633</div><div>|revision-id: squid3@treenet.co.nz-20160408073547-xtjco0mpai4lr920</div><div>|parent: chtsanti@users.sourceforge.net-20160407163610-yl8zzhc08l1ysv5f</div><div>|committer: Amos Jeffries <<a href="mailto:squid3@treenet.co.nz" target="_blank">squid3@treenet.co.nz</a>></div><div>|branch nick: trunk</div><div>|timestamp: Fri 2016-04-08 19:35:47 +1200</div><div>|message:</div><div>|  Fix shm_open error message after rev.14625</div><div>|------------------------------------------------------------</div><div>|# Bazaar merge directive format 2 (Bazaar 0.90)</div><div>|# revision_id: squid3@treenet.co.nz-20160408073547-xtjco0mpai4lr920</div><div>|# target_branch: <a href="http://bzr.squid-cache.org/bzr/squid3/trunk/" target="_blank">http://bzr.squid-cache.org/bzr/squid3/trunk/</a></div><div>|# testament_sha1: c1983f6601c29e4d03f936e60ecee221a860f932</div><div>|# timestamp: 2016-04-08 07:51:00 +0000</div><div>|# source_branch: <a href="http://bzr.squid-cache.org/bzr/squid3/trunk" target="_blank">http://bzr.squid-cache.org/bzr/squid3/trunk</a></div><div>|# base_revision_id: chtsanti@users.sourceforge.net-20160407163610-\</div><div>|#   yl8zzhc08l1ysv5f</div><div>|#</div><div>|# Begin patch</div><div>|=== modified file 'src/ipc/mem/Segment.cc'</div><div>|--- src/ipc/mem/Segment.cc     2016-04-03 23:41:58 +0000</div><div>|+++ src/ipc/mem/Segment.cc     2016-04-08 07:35:47 +0000</div><div>--------------------------</div><div>Patching file src/ipc/mem/Segment.cc using Plan A...</div><div>Hunk #1 failed at 91.</div><div>Hunk #2 succeeded at 105 (offset -4 lines).</div><div>Hunk #3 succeeded at 121 (offset -4 lines).</div><div>Hunk #4 succeeded at 151 (offset -1 lines).</div><div>1 out of 4 hunks failed--saving rejects to src/ipc/mem/Segment.cc.rej</div><div>Hmm...  The next patch looks like a unified diff to me...</div><div>The text leading up to this was:</div><div>--------------------------</div><div>|</div><div>|=== modified file 'src/ipc/mem/Segment.h'</div><div>|--- src/ipc/mem/Segment.h      2016-03-24 17:02:25 +0000</div><div>|+++ src/ipc/mem/Segment.h      2016-04-08 07:35:47 +0000</div><div>--------------------------</div><div>Patching file src/ipc/mem/Segment.h using Plan A...</div><div>Hunk #1 succeeded at 53.</div><div>Hmm...  Ignoring the trailing garbage.</div><div>done</div></div><div><br></div><div><br></div><div>Then when I compile, the compilation fails as follows (though without the patch the compilation succeeds):</div><div><br></div><div><div>libtool: compile:  clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\" -I../.. -I../../include -I../../lib -I../../src -I../../include -I/usr/include -I/usr/include -I../../libltdl -I/usr/include -I/usr/local/include/libxml2 -I/usr/local/include/libxml2 -Werror -Qunused-arguments -Wno-deprecated-register -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/Pages.lo -MD -MP -MF mem/.deps/Pages.Tpo -c mem/Pages.cc  -fPIC -DPIC -o mem/.libs/Pages.o</div><div>libtool: compile:  clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\" -I../.. -I../../include -I../../lib -I../../src -I../../include -I/usr/include -I/usr/include -I../../libltdl -I/usr/include -I/usr/local/include/libxml2 -I/usr/local/include/libxml2 -Werror -Qunused-arguments -Wno-deprecated-register -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/Pages.lo -MD -MP -MF mem/.deps/Pages.Tpo -c mem/Pages.cc -o mem/Pages.o >/dev/null 2>&1</div><div>depbase=`echo mem/PageStack.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`; /bin/sh ../../libtool  --tag=CXX    --mode=compile clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\"    -I../.. -I../../include  -I../../lib -I../../src  -I../../include  -I/usr/include  -I/usr/include  -I../../libltdl -I/usr/include -I/usr/local/include/libxml2  -I/usr/local/include/libxml2  -Werror -Qunused-arguments -Wno-deprecated-register  -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/PageStack.lo -MD -MP -MF $depbase.Tpo -c -o mem/PageStack.lo mem/PageStack.cc && mv -f $depbase.Tpo $depbase.Plo</div><div>libtool: compile:  clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\" -I../.. -I../../include -I../../lib -I../../src -I../../include -I/usr/include -I/usr/include -I../../libltdl -I/usr/include -I/usr/local/include/libxml2 -I/usr/local/include/libxml2 -Werror -Qunused-arguments -Wno-deprecated-register -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/PageStack.lo -MD -MP -MF mem/.deps/PageStack.Tpo -c mem/PageStack.cc  -fPIC -DPIC -o mem/.libs/PageStack.o</div><div>libtool: compile:  clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\" -I../.. -I../../include -I../../lib -I../../src -I../../include -I/usr/include -I/usr/include -I../../libltdl -I/usr/include -I/usr/local/include/libxml2 -I/usr/local/include/libxml2 -Werror -Qunused-arguments -Wno-deprecated-register -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/PageStack.lo -MD -MP -MF mem/.deps/PageStack.Tpo -c mem/PageStack.cc -o mem/PageStack.o >/dev/null 2>&1</div><div>depbase=`echo mem/Segment.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`; /bin/sh ../../libtool  --tag=CXX    --mode=compile clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\"    -I../.. -I../../include  -I../../lib -I../../src  -I../../include  -I/usr/include  -I/usr/include  -I../../libltdl -I/usr/include -I/usr/local/include/libxml2  -I/usr/local/include/libxml2  -Werror -Qunused-arguments -Wno-deprecated-register  -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/Segment.lo -MD -MP -MF $depbase.Tpo -c -o mem/Segment.lo mem/Segment.cc && mv -f $depbase.Tpo $depbase.Plo</div><div>libtool: compile:  clang++ -DHAVE_CONFIG_H -DDEFAULT_STATEDIR=\"/opt/squid-4/var/run/squid\" -I../.. -I../../include -I../../lib -I../../src -I../../include -I/usr/include -I/usr/include -I../../libltdl -I/usr/include -I/usr/local/include/libxml2 -I/usr/local/include/libxml2 -Werror -Qunused-arguments -Wno-deprecated-register -D_REENTRANT -g -O2 -march=native -std=c++11 -I/usr/local/include -MT mem/Segment.lo -MD -MP -MF mem/.deps/Segment.Tpo -c mem/Segment.cc  -fPIC -DPIC -o mem/.libs/Segment.o</div><div>mem/Segment.cc:96:22: error: too few arguments to function call, single argument 'err' was not specified</div><div>    if (!createFresh() && errno == EEXIST) {</div><div>         ~~~~~~~~~~~ ^</div><div>../../src/ipc/mem/Segment.h:56:5: note: 'createFresh' declared here</div><div>    bool createFresh(int &err);</div><div>    ^</div><div>mem/Segment.cc:98:21: error: too few arguments to function call, single argument 'err' was not specified</div><div>        createFresh();</div><div>        ~~~~~~~~~~~ ^</div><div>../../src/ipc/mem/Segment.h:56:5: note: 'createFresh' declared here</div><div>    bool createFresh(int &err);</div><div>    ^</div><div>mem/Segment.cc:108:9: error: use of undeclared identifier 'xerrno'</div><div>        xerrno = errno;</div><div>        ^</div><div>mem/Segment.cc:110:66: error: use of undeclared identifier 'xerrno'</div><div>        debugs(54, 5, "ftruncate " << theName << ": " << xstrerr(xerrno));</div><div>                                                                 ^</div><div>../../src/Debug.h:107:21: note: expanded from macro 'debugs'</div><div>            _dbo << CONTENT; \</div><div>                    ^</div><div>mem/Segment.cc:112:45: error: use of undeclared identifier 'xerrno'</div><div>               theName.termedBuf(), xstrerr(xerrno));</div><div>                                            ^</div><div>5 errors generated.</div><div>*** Error code 1</div><div><br></div><div>Stop.</div><div>make[3]: stopped in /usr/home/wash/ILI/Squid/4.x/squid-4.0.8/src/ipc</div><div>*** Error code 1</div><div><br></div><div>Stop.</div><div>make[2]: stopped in /usr/home/wash/ILI/Squid/4.x/squid-4.0.8/src</div><div>*** Error code 1</div><div><br></div><div>Stop.</div><div>make[1]: stopped in /usr/home/wash/ILI/Squid/4.x/squid-4.0.8/src</div><div>*** Error code 1</div><div><br></div><div>Stop.</div><div>make: stopped in /usr/home/wash/ILI/Squid/4.x/squid-4.0.8</div></div><div><br></div><div><br></div><div>Okay, I know I am being a bug here myself since I am focused on 3.5 now more than 4.x but I also like giving the little feedback I can from these FreeBSD servers I have:-)</div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 8 April 2016 at 17:19, Yuri Voinov <span dir="ltr"><<a href="mailto:yvoinov@gmail.com" target="_blank">yvoinov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA256<br>
<br>
</span>Seems fixed.<br>
<br>
<a href="http://bugs.squid-cache.org/show_bug.cgi?id=4486#c2" rel="noreferrer" target="_blank">http://bugs.squid-cache.org/show_bug.cgi?id=4486#c2</a><br>
<br>
08.04.16 18:05, Amos Jeffries пишет:<br>
<div><div class="h5">> On 8/04/2016 10:28 p.m., Odhiambo Washington wrote:<br>
>> Hello Yuri,<br>
>><br>
>> Thanks, but this patch is for squid-4.0.8, right??<br>
>><br>
><br>
> Yes the patch in that bug report is specific to the latest 4.0 snapshot,<br>
> which Yuri is running.<br>
><br>
> As Alex mentioned the errno usage in Squid was a bit wrong. The latest<br>
> 4.0 have a patch to fix that, which is causing Yuri's problem. Your 3.5<br>
> issue may or may not be related. But we wont know until after Yuri<br>
> applies that patch and checks the results.<br>
><br>
> Amos<br>
><br>
> _______________________________________________<br>
> squid-users mailing list<br>
> <a href="mailto:squid-users@lists.squid-cache.org">squid-users@lists.squid-cache.org</a><br>
> <a href="http://lists.squid-cache.org/listinfo/squid-users" rel="noreferrer" target="_blank">http://lists.squid-cache.org/listinfo/squid-users</a><br>
<br>
</div></div><span class="">-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
<br>
</span>iQEcBAEBCAAGBQJXB74GAAoJENNXIZxhPexGwkwH/ipdtdS23EuZHIQHWEN7DeSM<br>
aJ+b1lhRb+052wbL1vx8uyvhwGf++QVZcvFTGcxwFN0yQ1EaN1b1tE6IYlosG8IC<br>
1rCOkARyZnyLK0MZKqvajrv4RZMR9CAvcRspv7DftQwS/2/h2uhnakc2PZYKBSUO<br>
eF+hkxjUq2bSySY0sQ6tMTCr/U5x9IKzpg21VEE2tnn2S1HujnAEQ/C7pyi1ws7U<br>
SXyqQBOVr25wxQq3AZMg5Sk6uZ7pSF02soCaIon7SeBdx6K+eWTngg9/t1Rn5zit<br>
EGSrVLdEHsVFID/y5PsDMylpy4TwL+m0VCWSPbwYDGM+rp3tikfU/XAorFMR79M=<br>
=mVL9<br>
-----END PGP SIGNATURE-----<br>
<br>
<br>_______________________________________________<br>
squid-users mailing list<br>
<a href="mailto:squid-users@lists.squid-cache.org">squid-users@lists.squid-cache.org</a><br>
<a href="http://lists.squid-cache.org/listinfo/squid-users" rel="noreferrer" target="_blank">http://lists.squid-cache.org/listinfo/squid-users</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Best regards,<br>Odhiambo WASHINGTON,<br>Nairobi,KE<br>+254 7 3200 0004/+254 7 2274 3223<br>"<span style="font-size:12.8px">Oh, the cruft.</span><span style="font-size:12.8px">"</span></div></div></div>
</div>