Forum OpenACS Development: Project Manager from HEAD and current Repository version

Collapse
1: Project Manager from HEAD and current Repository version
Posted by Richard Hamilton on 07/22/09 08:50 AM
I downloaded the latest PM from cvs and the index page was throwing the following error in the Projects block of the page:

Error in include template "/web/webteamworks/packages/project-manager/lib/projects": invalid command name "ds_comment"

Turns out that there is a missing 'set' at the beginning of the line that declared the variable at the bottom of the projects.tcl file.

If I have commit I would commit the change but I don't! 😊

R.

Collapse
2: Re: Project Manager from HEAD and current Repository version (response to 1)
Posted by Derick Leony on 07/22/09 10:08 AM
ds_comment is a proc defined in the Developer Support package. The proper fix to this bug would be to check if acs-developer-support is installed before calling the proc.
Collapse
3: Re: Project Manager from HEAD and current Repository version (response to 1)
Posted by Richard Hamilton on 07/23/09 12:12 AM
Thanks, it isn't.

In which case the bug is that developer support should be a dependency for the PM package.

Many thanks
Richard

Collapse
4: Re: Project Manager from HEAD and current Repository version (response to 3)
Posted by Dave Bauer on 07/23/09 03:32 AM
I don't think so. I don't understand why DS code is in any application package. It should either 1) check is DS is installed or 2) Not call developer support at all. I suspect some programmer left this in for debugging and it is unecessary.
Collapse
5: Re: Project Manager from HEAD and current Repository version (response to 4)
Posted by Richard Hamilton on 07/23/09 11:23 AM
I'll remove it and see what happens! 😊

R.