Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 217472 - sci-chemistry/ccp4: wrong variable definition in install.sh script
Summary: sci-chemistry/ccp4: wrong variable definition in install.sh script
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Justin Lecher (RETIRED)
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2008-04-12 23:26 UTC by emsi
Modified: 2010-02-07 10:29 UTC (History)
2 users (show)

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


Attachments
Fixes to the setenv sed (ccp4-6.0.2-r1-setenv.ebuild.patch,824 bytes, patch)
2009-02-27 00:09 UTC, Johan Hattne
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description emsi 2008-04-12 23:26:15 UTC
the line:
setenv CINCL            $CCP4$CCP4/share/ccp4/include
in 
/etc/profile.d/ccp4.setup.sh
is wrong.
should be:
setenv CINCL            $CCP4/share/ccp4/include

Reproducible: Always

Steps to Reproduce:
run any CCP4 program in ebuild 6.0.2-r1

Actual Results:  
$ areaimol
>>>>>> CCP4 library signal ccp4_general:Cannot open environ.def (Success)
         raised in ccp4fyp <<<<<<
 areaimol:  Cannot open environ.def
 areaimol:  Cannot open environ.def
Times: User:       0.0s System:    0.0s Elapsed:     0:00


Expected Results:  
$ ncont

 ###############################################################
 ###############################################################
 ###############################################################
 ### CCP4 6.0: NCONT              version 6.0       :         ##
 ###############################################################
 User: marcin_cieslik  Run date: 13/ 4/2008 Run time: 00:49:31


 Please reference: Collaborative Computational Project, Number 4. 1994.
 "The CCP4 Suite: Programs for Protein Crystallography". Acta Cryst. D50, 760-763.
 as well as any specific reference in the program write-up.


 ------------------------------------------------------------------------------

 ***** ERROR #40 READ:

 Logical file name not found.
Comment 1 Donnie Berkholz (RETIRED) gentoo-dev 2008-04-14 04:17:48 UTC
Huh. Wonder how that slipped in. The stuff I run through ccp4i worked fine. I guess I'll collect a couple other things and make another revbump, since this seems like it could be pretty annoying for a lot of people.
Comment 2 Johan Hattne 2009-02-27 00:09:40 UTC
Created attachment 183315 [details, diff]
Fixes to the setenv sed

Just to make sure it isn't missed, CLIBD has the same problem as CINCL.  I attach a patch to the ebuild, which may or may not be the preferred way of fixing the issue.
Comment 3 Justin Lecher (RETIRED) gentoo-dev 2009-02-27 07:41:25 UTC
Thanks for that, I will fix it directly in the new package. Poke donnie for the tree ebuild. 
Comment 4 Justin Lecher (RETIRED) gentoo-dev 2010-02-07 10:29:06 UTC
I pushed version 6.1.3 to the tree. It should be fixed there