Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 454348 - >=app-doc/doxygen-1.8.1 segfaults on documentation of =media-libs/libmp3splt-0.8.1
Summary: >=app-doc/doxygen-1.8.1 segfaults on documentation of =media-libs/libmp3splt-...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Kacper Kowalik (Xarthisius) (RETIRED)
URL: https://sourceforge.net/p/mp3splt/bug...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-27 22:57 UTC by Sebastian Pipping
Modified: 2013-01-28 20:11 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Pipping gentoo-dev 2013-01-27 22:57:22 UTC
There might be a helpful patch in Debian.  For more details, please see the upstream bug report:

https://sourceforge.net/p/mp3splt/bugs/146/
Comment 1 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2013-01-28 08:15:36 UTC
+  28 Jan 2013; Kacper Kowalik <xarthisius@gentoo.org> +doxygen-1.8.3.1.ebuild,
+  +files/doxygen-1.8.3.1-empty-line-sigsegv.patch, -doxygen-1.8.1.2.ebuild,
+  -doxygen-1.8.2.ebuild:
+  Version bump including fixes for #454348 by Sebastian Pipping
+  <sping@gentoo.org>. Drop old

Please try and see if it fixes your issue
Comment 2 Sebastian Pipping gentoo-dev 2013-01-28 14:11:55 UTC
(In reply to comment #1)
> Please try and see if it fixes your issue

Works like a charm.  Thank you!


> +  28 Jan 2013; Kacper Kowalik <xarthisius@gentoo.org>
> +doxygen-1.8.3.1.ebuild,
> +  +files/doxygen-1.8.3.1-empty-line-sigsegv.patch, -doxygen-1.8.1.2.ebuild,
> +  -doxygen-1.8.2.ebuild:
> +  Version bump including fixes for #454348 by Sebastian Pipping
> +  <sping@gentoo.org>. Drop old

Actually the patch is not mine.  Please update the change log accordingly.  Thanks!
Comment 3 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2013-01-28 15:59:35 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > Please try and see if it fixes your issue
> 
> Works like a charm.  Thank you!
> 
> 
> > +  28 Jan 2013; Kacper Kowalik <xarthisius@gentoo.org>
> > +doxygen-1.8.3.1.ebuild,
> > +  +files/doxygen-1.8.3.1-empty-line-sigsegv.patch, -doxygen-1.8.1.2.ebuild,
> > +  -doxygen-1.8.2.ebuild:
> > +  Version bump including fixes for #454348 by Sebastian Pipping
> > +  <sping@gentoo.org>. Drop old
> 
> Actually the patch is not mine.  Please update the change log accordingly. 
> Thanks!
Changelog doesn't contain any information that you're the author of the patch. It's only noted that you were the reporter of bug #454348. Author of the patch is given credits in a proper place i.e. header of the patch along with information about debian and this bug report and what it fixes. Following "clean patch" guidelines by Mike[1]
Thanks for testing!

[1] http://dev.gentoo.org/~vapier/clean-patches
Comment 4 Sebastian Pipping gentoo-dev 2013-01-28 17:33:42 UTC
(In reply to comment #3)
> > > +  28 Jan 2013; Kacper Kowalik <xarthisius@gentoo.org>
> > > +doxygen-1.8.3.1.ebuild,
> > > +  +files/doxygen-1.8.3.1-empty-line-sigsegv.patch, -doxygen-1.8.1.2.ebuild,
> > > +  -doxygen-1.8.2.ebuild:
> > > +  Version bump including fixes for #454348 by Sebastian Pipping
> > > +  <sping@gentoo.org>. Drop old
> > 
> > Actually the patch is not mine.  Please update the change log accordingly. 
> > Thanks!
> Changelog doesn't contain any information that you're the author of the
> patch.

The current "fixes for X by Y" is ambiguous if "by Y" refers to "fixes" or to "X", with a tendency towards X since that's in proximity if you ask me.  If it were "fixes for X reported by Y" there were no ambiguity.  I'd welcome an update in that direction.
Comment 5 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2013-01-28 19:18:10 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > > > +  28 Jan 2013; Kacper Kowalik <xarthisius@gentoo.org>
> > > > +doxygen-1.8.3.1.ebuild,
> > > > +  +files/doxygen-1.8.3.1-empty-line-sigsegv.patch, -doxygen-1.8.1.2.ebuild,
> > > > +  -doxygen-1.8.2.ebuild:
> > > > +  Version bump including fixes for #454348 by Sebastian Pipping
> > > > +  <sping@gentoo.org>. Drop old
> > > 
> > > Actually the patch is not mine.  Please update the change log accordingly. 
> > > Thanks!
> > Changelog doesn't contain any information that you're the author of the
> > patch.
> 
> The current "fixes for X by Y" is ambiguous if "by Y" refers to "fixes" or
> to "X", with a tendency towards X since that's in proximity if you ask me. 
> If it were "fixes for X reported by Y" there were no ambiguity.  I'd welcome
> an update in that direction.

To properly fix that you'd need to retroactively adjust my (and several other people's) commits from past 2 years. Nevertheless:
http://sources.gentoo.org/viewvc.cgi/gentoo-x86/app-doc/doxygen/ChangeLog?r1=1.233&r2=1.234
Comment 6 Sebastian Pipping gentoo-dev 2013-01-28 20:11:43 UTC
(In reply to comment #5)
> http://sources.gentoo.org/viewvc.cgi/gentoo-x86/app-doc/doxygen/
> ChangeLog?r1=1.233&r2=1.234

Appreciated, thanks.