Summary: | lxde-base/lxtask - sandbox violation in /codeset.m4 by rm -f /codeset.m4 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Bryan Green <bgreen0> |
Component: | Current packages | Assignee: | LXDE team (DEFUNCT) <lxde+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | johanjanezer9, nullishzero |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build log
Patched lxtask Ebuild Diff Sandbox Patch |
Description
Bryan Green
2013-09-18 22:13:54 UTC
Created attachment 358966 [details]
build log
*** Bug 485410 has been marked as a duplicate of this bug. *** Created attachment 359050 [details]
Patched lxtask
Here is the fix. The tar archive contains the ebuild and 2 patches; one for adding AC_CONFIG_MACRO_DIR([m4]) to configure.ac and one for adding ACLOCAL_AMFLAGS= -I m4 to Makefile.am.
I'll also contact upstream with the fix.
Any feedback is welcome.
(In reply to Pavel Kazakov from comment #3) > Created attachment 359050 [details] > Patched lxtask > > Here is the fix. The tar archive contains the ebuild and 2 patches; one for > adding AC_CONFIG_MACRO_DIR([m4]) to configure.ac and one for adding > ACLOCAL_AMFLAGS= -I m4 to Makefile.am. > > I'll also contact upstream with the fix. > > Any feedback is welcome. -r1 is preferred compared to _p1 because this is a Gentoo revision and not an upstream patchset on top of 0.1.4 It would also be nice to combine both patches into a single patch since both aim to fix the same problem. (In reply to Markos Chandras from comment #4) > (In reply to Pavel Kazakov from comment #3) > > Created attachment 359050 [details] > > Patched lxtask > > > > Here is the fix. The tar archive contains the ebuild and 2 patches; one for > > adding AC_CONFIG_MACRO_DIR([m4]) to configure.ac and one for adding > > ACLOCAL_AMFLAGS= -I m4 to Makefile.am. > > > > I'll also contact upstream with the fix. > > > > Any feedback is welcome. > > -r1 is preferred compared to _p1 because this is a Gentoo revision and not > an upstream patchset on top of 0.1.4 > > It would also be nice to combine both patches into a single patch since both > aim to fix the same problem. Oh and please try to attach files (and ebuild diffs instead of complete ebuilds) instead of tarballs. Makes the review much easier and faster ;) Created attachment 359074 [details, diff] Ebuild Diff > Oh and please try to attach files (and ebuild diffs instead of complete > ebuilds) instead of tarballs. Makes the review much easier and faster ;) Thanks for the feedback :) Here's round 2. Attaching ebuild diff. Created attachment 359078 [details, diff]
Sandbox Patch
Attaching patch
Any luck with contacting upstream about that? +*lxtask-0.1.4-r1 (23 Sep 2013) + + 23 Sep 2013; Markos Chandras <hwoarang@gentoo.org> + +files/lxtask-0.1.4-sandbox.patch, +lxtask-0.1.4-r1.ebuild: + Add patch to fix sandbox violations. Bug #485348. Thanks to Pavel Kazakov + <nullishzero@gmail.com> + (In reply to Markos Chandras from comment #8) > Any luck with contacting upstream about that? Was waiting for the final go-ahead before sending it upstream. Issue has been noted on the upstream bugtracker[1]. 1) http://sourceforge.net/tracker/?func=detail&aid=3615048&group_id=180858&atid=894869 |