Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 231605 - bcpp does not read its own conf file in /etc/bcpp... here's a patch
Summary: bcpp does not read its own conf file in /etc/bcpp... here's a patch
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-12 17:23 UTC by Shawn Rutledge
Modified: 2008-12-17 16:56 UTC (History)
1 user (show)

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


Attachments
make bcpp try to read its conf file from /etc/bcpp/ (bcpp-20050725-etc-cfg-files.patch,675 bytes, patch)
2008-07-12 17:24 UTC, Shawn Rutledge
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn Rutledge 2008-07-12 17:23:32 UTC
The ebuild installs conf files in /etc/bcpp, but bcpp doesn't actually try to read them.  Instead it searches the system path.  (This is apparently a DOS relic)

Here's a patch to bcpp.cpp which tries to read the conf file from /etc/bcpp.  I would suggest applying it in the ebuild in the short term, and sending it upstream for the long term.


Reproducible: Always

Steps to Reproduce:
1.strace bcpp some-file.cpp 2>&1 | grep cfg
2.you will see it searching a lot of locations but not /etc/bcpp

Actual Results:  
doesn't find the conf file, produces default-formatted output

Expected Results:  
should have read the conf file that was supplied when bcpp was installed

patch will be attached, to be located in 

/usr/portage/dev-util/bcpp/files/bcpp-20050725-etc-cfg-files.patch
Comment 1 Shawn Rutledge 2008-07-12 17:24:27 UTC
Created attachment 160209 [details, diff]
make bcpp try to read its conf file from /etc/bcpp/
Comment 2 Carsten Lohrke (RETIRED) gentoo-dev 2008-07-13 22:11:31 UTC
Oh my - cpp team, any interest to maintain this little application. Otherwise forward assign to treecleaners, please.
Comment 3 Mark Loeser (RETIRED) gentoo-dev 2008-12-17 00:24:22 UTC
Treecleaners, please consider this for removal.
Comment 4 Shawn Rutledge 2008-12-17 02:01:17 UTC
Please don't remove it: it's useful
Comment 5 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-12-17 16:55:48 UTC
No sense in removing a working package that is usefull to someone. Patch applied and closing bug, thanks!

(amd64 stable at the same time)
(remove old version at same time)

%% strace ./bcpp some-file.cpp 2>&1 | grep cfg
open("bcpp.cfg", O_RDONLY)              = -1 ENOENT (No such file or directory)
open("/etc/bcpp/bcpp.cfg", O_RDONLY)    = -1 ENOENT (No such file or directory)