On Thu, Jun 16, 2011 at 11:20 PM, Len Trigg <lenbok+mlj...@gmail.com> wrote: > Kurt Harriger wrote: >> I had problems getting jdee to work when color-theme was installed first. >> Both jdee and color-theme define a function named >> replace-in-string if not already defined but the implementation differs. >> http://bzr.savannah.gnu.org/lh/color-theme/trunk/annotate/head:/color-theme.el >> >> I renamed replace-in-string to jde-replace-in-string >> My changeset is on github here: >> https://github.com/kurtharriger/jdee > > Should this be a complete replacement? (Because after your changeset, > there is still a remaining reference to replace-in-string) >
Yes, it should have been not sure how I missed that one. https://github.com/kurtharriger/jdee/tree/replace-in-string-fix > >> I also added included ant-contrib jar as its not installed by default on OS >> X. > > I would think of this as something you should add to your own ant > install (e.g. in ~/.ant/lib/) rather than adding it to the jdee > repository. > I thought I had to put it in /usr/share/ant/lib and I avoid putting anything in my root filesystem that I don't absolutely have to. ~/.ant/lib is more palatable, but I still prefer to keep dependencies that are not explicitly versioned and downloaded automatically by the build in the project itself. Its up to you if you want to apply that patch however and I created a new branch that has just the changes for replace-in-string. https://github.com/kurtharriger/jdee/tree/replace-in-string-fix > > Cheers, > Len. > ------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev _______________________________________________ jdee-devel mailing list jdee-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jdee-devel
