Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 35101 - vlnx don't set the right LDPATH environment variable
Summary: vlnx don't set the right LDPATH environment variable
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-12-04 16:46 UTC by Wagner Sartori Junior
Modified: 2004-07-26 18:23 UTC (History)
2 users (show)

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


Attachments
vlnx-416e-r1.ebuild (vlnx-416e-r1.ebuild,906 bytes, text/plain)
2004-06-14 13:53 UTC, Cory Visi (RETIRED)
Details
uvscan.conf (uvscan.conf,103 bytes, text/plain)
2004-06-14 13:53 UTC, Cory Visi (RETIRED)
Details
uvscan.cron (uvscan.cron,79 bytes, text/plain)
2004-06-14 13:54 UTC, Cory Visi (RETIRED)
Details
uvscan.sh (uvscan.sh,43 bytes, text/plain)
2004-06-14 13:54 UTC, Cory Visi (RETIRED)
Details
check-updates.sh (check-updates.sh,1.59 KB, text/plain)
2004-06-14 13:55 UTC, Cory Visi (RETIRED)
Details
vlnx-416e-envd (vlnx-416e-envd,17 bytes, text/plain)
2004-06-14 13:56 UTC, Cory Visi (RETIRED)
Details
vlnx-432e.ebuild (vlnx-432e.ebuild,1.01 KB, text/plain)
2004-06-17 13:25 UTC, Cory Visi (RETIRED)
Details
vlnx-432e-envd (vlnx-432e-envd,17 bytes, text/plain)
2004-06-17 13:26 UTC, Cory Visi (RETIRED)
Details
Modified check-updates to use ncftpget instead of lftp (check-ncupdates.sh,1.67 KB, text/plain)
2004-06-23 12:38 UTC, Sri Gupta
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wagner Sartori Junior 2003-12-04 16:46:35 UTC
The /etc/env.d/40vlnx set the wrong ld variable.

It set to LD_LIBRARY_PATH, and have to be LDPATH.

Reproducible: Always
Steps to Reproduce:
1.emerge vlnx
Actual Results:  
Editing the file, correct the problem
Comment 1 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:52:08 UTC
I can confirm this 100%. The env.d file is broken. In fact, you don't even need the PATH= (let's keep our PATHs clean).

I would like to suggest a makeover for this ebuild. Attached is all my stuff.
Comment 2 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:53:17 UTC
Created attachment 33243 [details]
vlnx-416e-r1.ebuild
Comment 3 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:53:57 UTC
Created attachment 33244 [details]
uvscan.conf
Comment 4 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:54:24 UTC
Created attachment 33245 [details]
uvscan.cron
Comment 5 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:54:50 UTC
Created attachment 33246 [details]
uvscan.sh
Comment 6 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:55:45 UTC
Created attachment 33247 [details]
check-updates.sh

If you'd like me to mirror this so it can be placed in SRC_URI, let me know.
Comment 7 Cory Visi (RETIRED) gentoo-dev 2004-06-14 13:56:34 UTC
Created attachment 33248 [details]
vlnx-416e-envd
Comment 8 Cory Visi (RETIRED) gentoo-dev 2004-06-14 14:03:41 UTC
One final thing, I believe check-updates.sh forces us to add:

RDEPEND="dev-lang/perl"

since it uses perl to massage the version string for virus definition updates.
Comment 9 Cory Visi (RETIRED) gentoo-dev 2004-06-14 14:08:20 UTC
I keep forgetting to mention... this is now under app-antivirus.
Comment 10 Cory Visi (RETIRED) gentoo-dev 2004-06-17 13:25:11 UTC
Created attachment 33439 [details]
vlnx-432e.ebuild

Woah! uvscan went berserk on me today on all my servers, eating up CPU or
segfaulting. This prompted me to upgrade to a newer version, which seems to
have fixed the problem. Attached are two new files.

I added a helpful info line for amavisd users, too.
Comment 11 Cory Visi (RETIRED) gentoo-dev 2004-06-17 13:26:55 UTC
Created attachment 33440 [details]
vlnx-432e-envd
Comment 12 Sri Gupta 2004-06-23 12:38:40 UTC
Created attachment 33969 [details]
Modified check-updates to use ncftpget instead of lftp

lftp hangs on "Resolving Host Address" for me, so I altered the script to use
ncftpget instead.  Otherwise identical.  Use if or as you like.
Comment 13 Cory Visi (RETIRED) gentoo-dev 2004-06-23 15:11:29 UTC
I have a feeling that I should be using wget so I do not have to worry about another dependency. In any case, I'll worry about that after I get some more feedback.
Comment 14 Cory Visi (RETIRED) gentoo-dev 2004-06-24 12:39:52 UTC
Do NOT use --fam in uvscan.conf. I just severely messed up my data partitions. I will update the uvscan.conf attachment as soon as I can.
Comment 15 Cory Visi (RETIRED) gentoo-dev 2004-06-28 21:51:42 UTC
vlnx-432e now in portage; please test :)
Comment 16 Wagner Sartori Junior 2004-06-29 06:31:40 UTC
the check-updates.sh script is compressed with gz. so, you have to gunzip it in the ebuild to use it!

very good work!

tkz
Comment 17 Cory Visi (RETIRED) gentoo-dev 2004-06-29 10:03:58 UTC
Woops! The unpack transparently handles the gunzip, but I had the doins command wrong. Nice find; change is in portage. And I bumped it to -r1 so everyone gets the script.
Comment 18 Wagner Sartori Junior 2004-06-30 06:11:57 UTC
I see that the cron.daily/uvscan, don't have the path in the uvscan line! I think that you can put the path to the cron.daily doesn't depend on PATH environment variable!

(my english is not good) very good work
Comment 19 Wagner Sartori Junior 2004-07-20 07:23:27 UTC
I think that the readme.txt don't show the version now. So, the check-update.sh file don't work as expected. So, I put another regular expression to a webpage, and everything works well. Can you update the file?

${WGET} "http://vil.nai.com/vil/DATReadme.asp"
if [ -f DATReadme.asp ]; then
        DAT_NEW=`perl -ne 'print $1 if /<td class="tableDataBackground">(\d{4})<\/td>/' DATReadme.asp`
else
        echo "Failed to retreive virus definitions version"
        rm -rf {$TEMPDIR}
        exit
fi

tkz...

Wagner Sartori Junior
Comment 20 Cory Visi (RETIRED) gentoo-dev 2004-07-20 07:37:47 UTC
Wagner, thank you for the contribution. I noticed this issue and I'm working on a fix.

In the future, please try not to reopen bugs unless you are certain the same exact bug is still occuring. In this case, this specific bug has almost nothing to do with the original bug.

I am closing this bug again. If you would like to open another bug, please do so, but I am working on a fix and should have it available soon.
Comment 21 Cory Visi (RETIRED) gentoo-dev 2004-07-26 18:23:50 UTC
Ok Wagner, check out the version in CVS now. I bumped the version, switched to the ZIP download (significantly smaller), now use update.ini to check the version, and fixed the path in the cron script.