Forum OpenACS Q&A: Error in my OpenACS install

Collapse
Posted by Peter Lim on

I've installed OpenACS on Fedora Core 2 using the automated install script, and the following error appears in the error.log.

Is this error important? How do I fix it if it's going to be a problem?

TIA,
Peter

>>>>>>>>>>
Error: Error sourcing  /home/httpd/service0/packages/acs-content-repository/tcl/content-permission-procs.tcl: 
can't create procedure  "content::revision::revoke_permission_h__arg_parser": unknown namespace
    while executing
"proc content::revision::revoke_permission_h__arg_parser {} {    ::upvar args args

    ::for { ::set i 0 } { $i < [::llength $args] } { ::incr i } {
 ..."
    ("uplevel" body line 1)
    invoked from within
"uplevel [::list proc ${proc_name_as_passed}__arg_parser {} $parser_code]"
    (procedure "ad_proc" line 304)
    invoked from within
"ad_proc -public content::revision::revoke_permission_h {
    -object_id:required 
    -revokee_id:required
    -privilege:required
} {
    @param objec..."
    (file "/home/httpd/service0/packages/acs-content-repository/tcl/content-permission-procs.tcl" line 172)
    invoked from within
"source $__file "
>>>>>
Collapse
Posted by Randy O'Meara on
Peter,

From what I can see, I believe that content::revision::revoke_permission_h should be content::permission::revoke_permission_h. Check line 172 of /packages/acs-content-repository/tcl/content-permission-procs.tcl and see what it looks like.

You must be pulling from CVS HEAD. I can't imagine this error cropping up on the released branch...

/R

Collapse
Posted by Peter Lim on
Randy,

That did it all right.  And yes, the automated install script must pull from CVS since the latest tarball is 5.1.0, and my OpenACS install says that it's running 5.2.0d4.

Thanks for the quick fix!

Peter

Collapse
Posted by Randy O'Meara on
Great. Now, can someone with commit privs look at this problem on HEAD?

Peter, you can change the oacs_branch (from memory) variable in install.tcl to pull from the oacs-5-1 branch, which you should probably do unless you really need to work in the unstable HEAD environment.

/R

Collapse
Posted by Jeff Davis on
The error isn't important (the function is one of the new ones added for the CR Tcl API by Dave Bauer). It's only on HEAD and is now fixed. Thanks for tracking it down.
Collapse
Posted by Randy O'Meara on
Thank you for the quick fix, Jeff.