<div dir="ltr"><div>&gt;It is something we should fix. If there is an update of the .spec<br>
needed, a correction like this can get included. Otherwise just follow<br>
the normal process for a bugfix (file a bug, post a patch).<br>&gt;<br><br>Yep, this is fixed in master (<a href="http://review.gluster.org/#/c/8116/">http://review.gluster.org/#/c/8116/</a>) .  <br>We have to include it in 3.6 branch. May be with logrotate spec file patch?<br><br></div>--Humble <br></div><div class="gmail_extra"><br clear="all"><div><div dir="ltr"><div>--Humble<br></div><a href="http://humblec.com" target="_blank">http://humblec.com</a><br></div></div>
<br><div class="gmail_quote">On Wed, Oct 29, 2014 at 1:22 AM, Niels de Vos <span dir="ltr">&lt;<a href="mailto:ndevos@redhat.com" target="_blank">ndevos@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Oct 29, 2014 at 01:01:28AM +0530, Lalatendu Mohanty wrote:<br>
&gt; On 10/28/2014 11:32 PM, Anders Blomdell wrote:<br>
&gt; &gt;Does the following warning:<br>
&gt; &gt;   warning: bogus date in %changelog: Thu Jun 29 2014 Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
&gt; &gt;warrant a bug report/fix as below (based on date of commit 8d8abc19)? Or should I just leave it as is?<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;diff --git a/<a href="http://glusterfs.spec.in" target="_blank">glusterfs.spec.in</a> b/<a href="http://glusterfs.spec.in" target="_blank">glusterfs.spec.in</a><br>
&gt; &gt;index f373b45..5ad4956 100644<br>
&gt; &gt;--- a/<a href="http://glusterfs.spec.in" target="_blank">glusterfs.spec.in</a><br>
&gt; &gt;+++ b/<a href="http://glusterfs.spec.in" target="_blank">glusterfs.spec.in</a><br>
&gt; &gt;@@ -1042,7 +1042,7 @@ fi<br>
&gt; &gt;  * Wed Sep 24 2014 Balamurugan Arumugam &lt;<a href="mailto:barumuga@redhat.com">barumuga@redhat.com</a>&gt;<br>
&gt; &gt;  - remove /sbin/ldconfig as interpreter (#1145992)<br>
&gt; &gt;-* Thu Jun 29 2014 Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
&gt; &gt;+* Thu Jun 19 2014 Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
&gt; &gt;  - Added dynamic loading of fuse module with glusterfs-fuse package installation in el5.<br>
&gt; &gt;  * Fri Jun 27 2014 Kaleb S. KEITHLEY &lt;<a href="mailto:kkeithle@redhat.com">kkeithle@redhat.com</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt;/Anders<br>
&gt; I checked master branch and I can see the date and day is correct in<br>
&gt; <a href="http://glusterfs.spec.in" target="_blank">glusterfs.spec.in</a> as below. Where do you see wrong change log i.e. Thu Jun<br>
&gt; 29 2014 Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;?<br>
&gt;<br>
&gt; * Thu Jun 19 2014 Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
&gt; - Added dynamic loading of fuse module with glusterfs-fuse package<br>
&gt; installation in el5.<br>
<br>
</span>That seems to come from commit 8d8abc19 in release-3.6:<br>
<br>
commit 8d8abc19b0500b725f92164c096b64b3c37cd6b0<br>
Author: Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
Date:   Thu Jun 19 16:51:57 2014 +0530<br>
<br>
    Autoload fuse module in centos5/el5 versions.<br>
<br>
    This patch dynamically load &#39;fuse&#39; module with the installation<br>
    of glusterfs-fuse package in centos/el5 distributions.<br>
<br>
    Change-Id: I4988a025bd18716fd6d355431196540e140224b3<br>
    BUG: 1108669<br>
    Signed-off-by: Humble Chirammal &lt;<a href="mailto:hchiramm@redhat.com">hchiramm@redhat.com</a>&gt;<br>
    Reviewed-on: <a href="http://review.gluster.org/8116" target="_blank">http://review.gluster.org/8116</a><br>
    Reviewed-by: Niels de Vos &lt;<a href="mailto:ndevos@redhat.com">ndevos@redhat.com</a>&gt;<br>
    Reviewed-by: Lalatendu Mohanty &lt;<a href="mailto:lmohanty@redhat.com">lmohanty@redhat.com</a>&gt;<br>
    Tested-by: Gluster Build System &lt;<a href="mailto:jenkins@build.gluster.com">jenkins@build.gluster.com</a>&gt;<br>
<br>
<br>
It is something we should fix. If there is an update of the .spec<br>
needed, a correction like this can get included. Otherwise just follow<br>
the normal process for a bugfix (file a bug, post a patch).<br>
<br>
Thanks,<br>
Niels<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org">Gluster-devel@gluster.org</a><br>
<a href="http://supercolony.gluster.org/mailman/listinfo/gluster-devel" target="_blank">http://supercolony.gluster.org/mailman/listinfo/gluster-devel</a><br>
</div></div></blockquote></div><br></div>