Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 412209 - app-arch/p7zip - optionally build 7zFM, the 7zip file manager
Summary: app-arch/p7zip - optionally build 7zFM, the 7zip file manager
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Unclassified (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Justin Lecher (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-16 12:34 UTC by shad0VV
Modified: 2012-04-23 20:21 UTC (History)
0 users

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 shad0VV 2012-04-16 12:34:38 UTC
p7zip includes 7zFM,which is the gui for p7zip.I need a maintainer to make the right changes to the current ebuild in order to include 7zFM.

I compared the current ebuild and the makefile of p7zip,the maintainer uses emake all3 which doesn't include 7zFM the right one is emake all4.I am not sure if this is the only change ebuild needs.

[Ebuild]
src_compile() {
 	emake all3 || die "compilation error"
 	if use kde || use wxwidgets; then
 	emake 7zG || die "error building GUI"
 	fi
	}
[Makefile of source code]
default:7za

all:7za sfx

all2: 7za sfx 7z

all3: 7za sfx 7z 7zr

all4: 7za sfx 7z 7zr Client7z 7zG 7zFM

all_test : test test_7z test_7zr test_Client7z
	$(MAKE) -C CPP/7zip/Compress/LZMA_Alone  test


Thanks in advance!
Comment 1 Franz Trischberger 2012-04-17 07:04:31 UTC
According to the code you posted, all4 will build 3 more targets, including Client7z and 7zG.
Instead of changing all3 to all4, you should just add
emake 7zFM || die "7zFM building failed"
But watch out, probably you need to make changes to src_insrall, too!
Comment 2 Justin Lecher (RETIRED) gentoo-dev 2012-04-23 20:21:33 UTC
fixed today.