Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 88103 - xawtv uses a hardcoded test in order to install a cgi file.
Summary: xawtv uses a hardcoded test in order to install a cgi file.
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High normal
Assignee: Television related Applications in Gentoo's Portage
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 88105
  Show dependency tree
 
Reported: 2005-04-05 17:46 UTC by Octavio Ruiz (Ta^3)
Modified: 2005-05-29 17:41 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Octavio Ruiz (Ta^3) 2005-04-05 17:46:05 UTC
Xawtv uses a hardcoded test in order to install a cgi file.

       if [ -d /var/www/localhost ]
       then
              ....
       fi

In could be not true, /srv/www/localhost is valid, or wherever the user wants.

To install just a .cgi file I think is too much inherit webapp.eclass, so -I think- leaving the webcam.cgi file inside /usr/share/doc/${PF}/DOCDESTTREE ist a good idea.

Something like

  docinto cgi
  dodoc scripts/webcam.cgi
Comment 1 Octavio Ruiz (Ta^3) 2005-04-05 18:09:46 UTC
See this attachments:

xawtv-3.94-r1.ebuild (webapp support):
http://bugs.gentoo.org/attachment.cgi?id=55411&action=view
xawtv-3.94-r1.patch (webapp support):
http://bugs.gentoo.org/attachment.cgi?id=55413&action=view
Comment 2 Octavio Ruiz (Ta^3) 2005-04-27 02:01:33 UTC
xawtv-3.94-r2: Now see Attachment #57366 [details] (ebuild) or Attachment #57638 [details] (diff)  Maybe its better, not to use webapp eclass. 

:-)
Comment 3 Octavio Ruiz (Ta^3) 2005-04-27 02:02:51 UTC
xawtv-3.94-r2: Now see Attachment #57366 [details] (ebuild) or Attachment #57368 [details, diff] (diff)  Maybe its better, not to use webapp eclass. 

:-)
Comment 4 Elfyn McBratney (beu) (RETIRED) gentoo-dev 2005-04-27 02:24:21 UTC
Hrmm, though you could hard-code /var/www/... in the ebuilds/perform the install yourself, using the webapp eclass has advantages of ease and good tools, and will make migrations to new data directories (/var/www) a hell of a lot easier, too.

Just my 0.02p. :)
Comment 5 Octavio Ruiz (Ta^3) 2005-04-27 10:36:42 UTC
Yep, but we are talking about a one single cgi file, and I think that calling webapp eclass just for an 60 lines cgi which just look for an image that /usr/bin/webcam leave on 'public_html/images' for make a nescape-only compatibly "image stream" its not neccesary. Also maybe it just not works as-is, or like the user wants. IMHO is better to leave it as documentation.

/usr/share/doc/xawtv-3.94-r2/webcamrc.gz
/usr/share/doc/xawtv-3.94-r2/cgi-bin
/usr/share/doc/xawtv-3.94-r2/cgi-bin/webcam.cgi

The point here, is that the current portage tree ebuild doesnt looks good. :-|

I propose two solutions, one is the webapp eclass and there is their "proof-of-concept" ebuild in Attachment 55411 [details] (not updated, but it should work) for it. I dont like it a lot because the reason that I say before (also in fact its not a www related package), and the other one is leave the webcamrc and webcam.cgi in the docs dir like in Attachment #57366 [details] (its updated)

:-)
Comment 6 Doug Goldstein (RETIRED) gentoo-dev 2005-05-29 17:41:03 UTC
Fixed in CVS.