[cvs] commit: framework/UI/UI/VarRenderer html.php

[Michael Rubinsky]

I guess where I disagree here is that I feel that CVS HEAD *is* Horde  
4 code - even if it hasn't been totally refactored, improved,  
whatever.  If people are running HEAD, they are running Horde 4 - and  
thus will need the git code.


Quoting Michael M Slusarz <slusarz at horde.org>:

> Quoting Chuck Hagenbuch <chuck at horde.org>:
>
>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>
>>> This isn't correct.  CVS code should not be using libraries
>>> contained in git.  Same goes for the turba commits.
>>
>> Given that we've been moving libraries from CVS to git, I'd say this
>> isn't true... What's behind your position?
>
> This was the whole reason I moved stuff like IMP and Chora to git -  
> to allow these packages to continue working for those doing CVS  
> updates without the need to do git setup (obviously, for IMP at  
> least, that's not entirely true because of the Horde_Imap_Client  
> move to git, but IMP cvs is totally unsupported and broken and this  
> point which is why I was pushing to remove it from CVS several weeks  
> ago).

I believe there have already been some framework libraries moved out  
of CVS HEAD into Git that were part of FW_3, so theoretically, it's  
possible that *any* HEAD code in CVS could be broken with a git  
checkout.

> The momentum to be moving stuff over to git should be coming from  
> the use of the new Horde 4 code.  Allowing use of this code in Horde  
> 3-ish proper sort of defeats the purpose.

I guess where I disagree here is that I feel that CVS HEAD *is* Horde  
4 code - even if it hasn't been totally refactored, improved,  
whatever.  If people are running HEAD, they are running Horde 4 - and  
thus will need the git code.

That being said, if using Git code in an application is grounds for  
moving it into Git, then I, personally, don't have any objections to  
moving the application. What I *don't* want though is to have to  
maintain a "HEAD" application in both CVS and Git.

-- 
Thanks
Mike

"Time just hates me, that's why it made me an adult." - Josh Joplin