Summary: | silc-client-0.7.6.2.ebuild (New Package) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Stein Magnus Jodal <stein.magnus> |
Component: | New packages | Assignee: | chouser (RETIRED) <chouser> |
Status: | VERIFIED FIXED | ||
Severity: | enhancement | CC: | stein.magnus |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | x86 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | silc-client-0.7.6.2.ebuild |
Description
Stein Magnus Jodal
2002-02-05 17:12:50 UTC
Created attachment 130 [details]
silc-client-0.7.6.2.ebuild
Here are some initial comments... You can submit a new version if you'd like, otherwise I will fix these when I get a chance. - please use a variable for at least the version of the package in SRC_URI - there is a 'ncurses' USE var; avoid making the user to edit the ebuild. - refer to ncurses USE var in DEPENDS as well - should be "use ipv6" not "user ipv6" - are all the configure settings needed? could be -- I haven't tested them yet - are all the make install settings needed? again, they might be - make install's docdir is missing the $(D) Some of these mistakes would have been easier for you to catch if you had looked at (or started with) /usr/portage/skel.ebuild instead of an existing and possibly older ebuild. Others would be caught by including "sandbox" in your MAINTAINER setting in /etc/make.conf (assuming you have an up to date version of portage), and then tested the ebuild. Thanks for the ebuild -- we appreciate it! --Chouser Everything mentioned fixed. That ncurses was a USE variable is neither mentioned in the guide nor in /etc/make.conf. user -> use should be fixed in the Irssi ebuild file too. Bugzilla won't let me attach a file, so I posted it at http://www.s-mag.net/temp/silc-client-0.7.6.2.ebuild Thanks for making the changes. I'm sorry; I was mistaken about the ncurses use variable -- thanks for checking the docs and correcting me. I've posted a bug for the Irssi ebuild (bug 567) This is a messy app to package, I'm afraid... Once I started testing the install, I found that it was defaulting to all sorts of bad places to put various files. I think I've fixed them all. I also added the 'experimental' socks5 USE var. I don't really know how to use this app, so if you could give this ebuild a good testing, I'd appreciate it. Also, I put this in the net-im category. Does that seem right? Thanks for the ebuild! It is not compatible with IRC, but it is used in the same way and the client is based on an IRC client. net-irc could be confusing, net-im should be okay since IRC is a form of instant messaging. |