[24915] branches/release_1_4/base

[Juan Manuel Palacios]

On May 8, 2007, at 1:32 PM, Daniel J. Luke wrote:

> On May 8, 2007, at 1:11 AM, source_changes at macosforge.org wrote:
>> Also, moving the IndexRegen.sh script to check out from the tags  
>> dir, thus keeping the checkout always in sync with the latest  
>> MacPorts release. NOTE to dluke: You should probably reinstall  
>> this script and bootstrap it again ;-)
>
> This change won't actually work the way you want it to.
>
> Currently, I manually move a copy of IndexRegen.sh into position to  
> actually run on my machine (as I don't particularly want anyone  
> with commit access to be able to run arbitrary shell scripts on my  
> machine (although it's probably a mute point as anyone with commit  
> access could modify any of the base/ code that gets executed just  
> as easily)).


	I understand taking precautions ;-)

>
> SVN_BASE_URL isn't actually used by the script for much more than  
> documentation, so changing it wont change where the checkout happens.


	Yeah, the script does not bootstrap the checkout, but only updates  
it (therefore, the checkout could happen from any URL, initially). In  
any case, I only meant updating the variable for documentation  
purposes too, to reflect what release we're indexing with. All this  
is, arguably, largely unnecessary, so we shouldn't be too stringent  
as long as it's always clear what release the script tracks  
(therefore, update from the current release branch, from a tag...  
doesn't make much difference as long as the intent is clear).

>
> The script pulls from the latest release branch, and therefore  
> doesn't need intervention unless/until major releases (when I would  
> manually switch the repo to the later branch). Changing to a tag  
> would require manual intervention on my part for every release.

	I understand wanting to minimize maintenance, so lets keep it  
pointing to the branch then.

>
> In fact, I would rather we went in the opposite direction, where  
> there was a 'current-release' branch that was used for the index  
> regen.


	Not sure what you mean by this.... what we just coincided on  
above...? Or are you referring to some alternate approach?


-jmpp


>
> --
> Daniel J. Luke
> +========================================================+
> | *---------------- dluke at geeklair.net ----------------* |
> | *-------------- http://www.geeklair.net -------------* |
> +========================================================+
> |   Opinions expressed are mine and do not necessarily   |
> |          reflect the opinions of my employer.          |
> +========================================================+
>
>