Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 10562 - chemtool-1.5.ebuild (new package)
Summary: chemtool-1.5.ebuild (new package)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: George Shapovalov (RETIRED)
URL: http://ruby.chemie.uni-freiburg.de/~m...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-11-10 20:53 UTC by Marshall T. Vandegrift
Modified: 2003-02-04 19:42 UTC (History)
1 user (show)

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


Attachments
chemtool-1.5 ebuild and patches (chemtool.tar.bz2,1.91 KB, application/octet-stream)
2002-11-10 20:54 UTC, Marshall T. Vandegrift
Details
chemtool-1.5.ebuild (chemtool-1.5.ebuild,2.19 KB, application/octet-stream)
2002-11-12 02:45 UTC, George Shapovalov (RETIRED)
Details
ChangeLog (ChangeLog,302 bytes, application/octet-stream)
2002-11-12 02:46 UTC, George Shapovalov (RETIRED)
Details
updated chemtool-1.5 ebuild and patches (chemtool.tar.bz2,2.92 KB, application/octet-stream)
2002-11-13 01:07 UTC, Marshall T. Vandegrift
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marshall T. Vandegrift 2002-11-10 20:53:19 UTC
Please find attached chemtool.tar.bz2, containing an ebuild for chemtool-1.5 and
various patches to help it build Gentoo-ly.

From chemtool's Web page:

  Chemtool is a program for drawing organic molecules

Filing it under app-sci/chemtool seems reasonable

--
marshall t. vandegrift (vandem2@rpi.edu)
Comment 1 Marshall T. Vandegrift 2002-11-10 20:54:53 UTC
Created attachment 5564 [details]
chemtool-1.5 ebuild and patches
Comment 2 George Shapovalov (RETIRED) gentoo-dev 2002-11-12 02:44:21 UTC
Hi Marshall.

Thank you for your submission!
Unfortunately there are few problems that have to be sorted out:

1. Please convert patches to a unified diff format (diff -u).

2. It is not very good to force C[XX]FLAGS to a strictly set value unless
absolutely necessary. More importantly, forcing them via patch is unnecessary
(the patch to Makefie.in). Please try to use the provided eclass: flag-o-matic.
You can take a look at the eclass itself for the definition and description of
all the provided functions. You can also grep through ebuilds for the examples
(filter-flags is the most often used function which is designed to remove some
problematic setting from C[XX]FLAGS).

3. The configure script apparently ignores --without-kdedir flag (it still
autodetects kde and sets the patch correspondingly). Something should be done
anout it.

4. In present form ebuild halts during src_install reporting SANDBOX_VIOLATIOn -
the is trying to write to the live filesystem instead of under ${D}. I corrected
 installation of locales, however the kde issue is not so easy. Could you please
polish it a bit? (In order to see sandbox violation complaints you need to have
"sandbox" in the FEATURES in your make.conf).

I am attaching the modified ebuild to the bug.

George
Comment 3 George Shapovalov (RETIRED) gentoo-dev 2002-11-12 02:45:57 UTC
Created attachment 5597 [details]
chemtool-1.5.ebuild

Modified ebuild chemtool-1.5.ebuild
Comment 4 George Shapovalov (RETIRED) gentoo-dev 2002-11-12 02:46:54 UTC
Created attachment 5598 [details]
ChangeLog

So that I don't loose it :).
Comment 5 Marshall T. Vandegrift 2002-11-13 01:07:29 UTC
Hi George,

Thanks for your feedback.

:-)

This ebuild represents the first (manual) step towards automating the process of
converting FreeBSD ports to Gentoo ebuilds (not that I want to plague Gentoo
with FreeBSD's ports problems...  It just seems like it might be useful). 
Anyway, if I'd been more careful, I would have started with a package not as
hopelessly b0rked as this one...

Pertinant to your specific comments:

1. All diffs now in unified format.  Sorry 'bout that.

2. My patches *stop* the Makefile.ins from hard-coding CFLAGS, so no problems.
:-)  On the other hand, thank you for the suggestion that I look at possibly
applicable eclasses...

3. Fixed.  Ugh.  My patches are almost complete re-writes of the affected files.
 As one bright point, it turns out that an appropriate KDE directory can be
autodetected using the 'need-kde' function in the 'kde-functions' eclass.

4. I do have sandbox turned on.  This only popped up if GNOME or KDE support
were enabled, which I just assumed would work (BAD IDEA, I know... I won't do it
in the future).

Whew.  Updated ebuild and patches in following attachment.

--
marshall t. vandegrift (vandem2@rpi.edu)
Comment 6 Marshall T. Vandegrift 2002-11-13 01:07:57 UTC
Created attachment 5629 [details]
updated chemtool-1.5 ebuild and patches
Comment 7 George Shapovalov (RETIRED) gentoo-dev 2002-11-28 00:11:39 UTC
Hi Marshall.

Thanks for the update, this looks good now ;).
I did few minor clean-ups and committed the ebuild. Please test.

George