Forum OpenACS Q&A: Re: Package Dependencies in OpenACS

Collapse
Posted by Orzenil Silva Junior on
Thank you, Nima, for your investigative work.

I tried find ui package in cvs looking at several branches and i think this package was really removed (or it never was initiate in cvs).

I really want try recruiting package for dotlrn so i find this cvs log in xarg.net -

http://xarg.net/tools/cvs/change-set-details?key=1497 -

showing packages/recruiting/tcl/table-sorter-procs.tcl was removed and table ui stuff was moved to new standalone ui package. But there are no logs for ui package commits.

I don't know it is a good idea but as i need a start point to a recruiting package i found an old file for table-sorter-procs.tcl in

http://cvs.openacs.org/cvs/openacs-4/packages/recruiting/tcl/Attic/table-sorter-procs.tcl?rev=1.1&view=markup

then I rename this file to packages/recruiting/tcl/recruiting-ui-procs.tcl and modified it to put recruiting installing on dotlrn-2-1 avoiding dependencies on ui package. It worked well so i uploaded the files i changed to

https://openacs.org/storage/file?file%5fid=243058

I did not create a patch for it yet because i think i could hear from people maintaining recruiting if ui package will be live in cvs.

Collapse
Posted by Jade Rubick on
Orzenil: if it's in the attic, it's either been removed, or it exists on a different branch. That could be the problem.
Collapse
Posted by Orzenil Silva Junior on
Jade: in fact table-sorter-procs file was really removed (as showed by http://cvs.openacs.org/cvs/openacs-4/packages/recruiting/tcl/Attic/table-sorter-procs.tcl?only_with_tag=HEAD) but recruiting and its dotlrn portlet and applet needs table:: from table-sorter to render tables for output then I just rewrite table-sorter-procs declaring new tcl namespace called ui. Maybe it is not necessary because i think acs-templating could deal with table-sorter-procs requirements so we could avoid ui package dependencies without reintroducing table-sorter-procs but i was in a hurry to get recruiting work just to see what it does.

Thank you for your advertisement.