Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 243266

Summary: New ebuild: for app-dics/dictconv
Product: Gentoo Linux Reporter: Behzat <behzaterte>
Component: New packagesAssignee: Default Assignee for New Packages <maintainer-wanted>
Status: RESOLVED DUPLICATE    
Severity: enhancement CC: behzaterte, tommy
Priority: High Keywords: EBUILD
Version: unspecified   
Hardware: AMD64   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: ebuild for app-dics/dictconv
Ebuild for /app-dicts/dictconv version 2
Ebuild dictconv-0.2 version three.
dictconv-0.2.ebuild

Description Behzat 2008-10-22 19:37:12 UTC
I write for app-dics/dictconv ebuild file. It's work for amd64 but I'dont success copying to /usr/bin directory.

Reproducible: Always
Comment 1 Behzat 2008-10-22 19:38:12 UTC
Created attachment 169470 [details]
ebuild for app-dics/dictconv
Comment 2 Behzat 2008-10-22 19:48:47 UTC
Created attachment 169476 [details]
Ebuild for /app-dicts/dictconv version 2

I changed some lines. Sorry for confusion my first ebuild file :-))
Comment 3 Thomas Sachau gentoo-dev 2008-10-22 20:17:18 UTC
Some comments on the ebuild:

-KEYWORDS should be in alphabetical order
-DEPEND="dev-libs/libxml2" not your way
-econf dies itself, does not need a || die
-Does emake install not install dictconv to ${D}/usr/bin? If not, use "dobin dictconv || die"
Comment 4 Rafał Mużyło 2008-10-22 21:05:23 UTC
Also this thing needs more than a little gcc 4.3 patching,
mainly cstring and cstdlib.
Comment 5 Rafał Mużyło 2008-10-22 21:09:45 UTC
As a side note: whoever wrote that
knew very little about autotools.
If it's really 0.2, configure.in should have been updated.
While it doesn't really matter for the program, it still looks bad.
Comment 6 Rafał Mużyło 2008-10-22 21:13:05 UTC
And what is exactly '--with-posix-regex' for ?
configure.in doesn't show anything about it.
Comment 7 Rafał Mużyło 2008-10-23 12:02:10 UTC
One more thing.
In src/Makefile.am, libs should be added to LDADD, not LDFLAGS.
For why's see 'info automake'.
Comment 8 Behzat 2008-10-24 17:57:45 UTC
Created attachment 169748 [details]
Ebuild dictconv-0.2 version three.
Comment 9 Thomas Sachau gentoo-dev 2008-10-24 19:35:29 UTC
I is probably DEPEND, not DPEND. RDEPEND can be dropped (some people like it to stay there). Also, what do you want to do in src_install???
Comment 10 Behzat 2008-10-24 19:59:20 UTC
(In reply to comment #9)
> I is probably DEPEND, not DPEND. RDEPEND can be dropped (some people like it 

Sorry, I missed this line. 

> Also, what do you want to do in src_install???

I just want to trying copy executable file to /usr/bin but may be I writed wrong also this ebuild my first. Sorry for confusion.


By the way what's your suggestion for copy?

Thanks.
Comment 11 Thomas Sachau gentoo-dev 2008-10-24 21:59:44 UTC
Created attachment 169766 [details]
dictconv-0.2.ebuild

This is what i would suggest with a short look on the ebuild. If you would like to discuss it further or maintain it yourself, you could join #gentoo-sunrise at irc.freenode.net or read https://overlays.gentoo.org/proj/sunrise to get some more information.
Comment 12 Behzat 2008-10-24 22:10:00 UTC
(In reply to comment #11)
> Created an attachment (id=169766) [edit]
> dictconv-0.2.ebuild
> 
> This is what i would suggest with a short look on the ebuild. If you would like
> to discuss it further or maintain it yourself, you could join #gentoo-sunrise
> at irc.freenode.net or read https://overlays.gentoo.org/proj/sunrise to get
> some more information.
> 

Thank you for your support. I'll join that channel now.
Regards,
Behzat.

Comment 13 Torsten Veller (RETIRED) gentoo-dev 2008-10-28 09:54:39 UTC
This is a dup of 224525. Why don't you search?
Comment 14 Behzat 2008-10-28 10:08:47 UTC
(In reply to comment #13)
> This is a dup of 224525. Why don't you search?
> 

Sorry for this, I just want to writing new new ebuild file(my first try)

Regards,
Behzat.
Comment 15 Thomas Sachau gentoo-dev 2008-10-28 19:35:51 UTC

*** This bug has been marked as a duplicate of bug 224525 ***