fast-update should handle -D

RESOLVED WONTFIX

Status

SeaMonkey
Build Config
--
enhancement
RESOLVED WONTFIX
17 years ago
12 years ago

People

(Reporter: bbaetz, Assigned: bbaetz)

Tracking

Trunk
Future
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

4.61 KB, patch
Details | Diff | Splinter Review
(Assignee)

Description

17 years ago
If you want to use fast-update on a tinderbox, it should accept -D. Ihave 
apatch for this (which is running on Brad on MozillaTest now).

It uses the given date for the timestamp file, so hours-to-go are still ok, but
trying to fast-update by going backwards won't really work.

I'll attach the diff for comments.
(Assignee)

Comment 1

17 years ago
Created attachment 70861 [details] [diff] [review]
patch

Comment 2

17 years ago
Comment on attachment 70861 [details] [diff] [review]
patch

sr=alecf
very cool! Can't wait to see this on tinderbox.
Attachment #70861 - Flags: superreview+
Comment on attachment 70861 [details] [diff] [review]
patch

r=cls even though I still think that using fast-update on tinderboxes (you can
be sure they'll be more) is going to put an unnecessary load on bonsai when
pulling from cvs-mirror is even faster than fast-update.  But whatever.
(Assignee)

Comment 4

17 years ago
Need to play with 132756 before doing this, I think. Maybe they're not related -
need to check.

-> 1.1
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1alpha
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.1alpha → Future

Comment 5

15 years ago
Created attachment 125051 [details] [diff] [review]
Patch v1.0.1

This patch is merged to trunk, creates a spawn_cvs subroutine, and fixes an
error where in one spot the previous patch wanted to update $dirlist instead of
$pdir, as it did previously.
Attachment #70861 - Attachment is obsolete: true
Product: Browser → Seamonkey
(Assignee)

Comment 6

12 years ago
WONTFIX. cvs server got upgraded a while back
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.