contrib/cvs-update.sh sets sticky dates

RESOLVED FIXED in Bugzilla 2.18

Status

()

Bugzilla
Bugzilla-General
P3
normal
RESOLVED FIXED
16 years ago
4 years ago

People

(Reporter: Jacob Steenhagen, Assigned: Jacob Steenhagen)

Tracking

2.13
Bugzilla 2.18
x86
Linux

Details

Attachments

(1 attachment)

884 bytes, patch
bbaetz
: review+
Jouni Heikniemi
: review+
Details | Diff | Splinter Review
(Assignee)

Description

16 years ago
Whenever bugzilla is updated via the cvs-update.sh script it sets stick dates on
all bugzilla files.  This has the downside that individual files can't be
updated (unless the -A tag is used) and changes can't be commited.

What the patch I'm about to upload does is changes it so '-A' gets used when
updating (to remove any sticky dates/tags).  It still logs the command w/the -D
so that a change can backed out of the running install.
(Assignee)

Comment 1

16 years ago
Created attachment 39126 [details] [diff] [review]
patch
(Assignee)

Updated

16 years ago
Keywords: patch
Target Milestone: --- → Bugzilla 2.16
Priority: -- → P3
(Assignee)

Comment 2

16 years ago
-> New Bugzilla Product
Assignee: tara → jake
Component: Bugzilla → Bugzilla-General
Keywords: review
Product: Webtools → Bugzilla
Version: Bugzilla 2.13 → 2.13
We are currently trying to wrap up Bugzilla 2.16.  We are now close enough to
release time that anything that wasn't already ranked at P1 isn't going to make
the cut.  Thus this is being retargetted at 2.18.  If you strongly disagree with
this retargetting, please comment, however, be aware that we only have about 2
weeks left to review and test anything at this point, and we intend to devote
this time to the remaining bugs that were designated as release blockers.
Target Milestone: Bugzilla 2.16 → Bugzilla 2.18
Comment on attachment 39126 [details] [diff] [review]
patch

This will make the logged date different to the real date if a checkin occurs
between the echo and the update -A. I guess thats unlikely to be a problem in
practice.

r=bbaetz if you think we're happy with that limitation
Attachment #39126 - Flags: review+

Comment 5

15 years ago
Comment on attachment 39126 [details] [diff] [review]
patch

r=jouni
Attachment #39126 - Flags: review+
myk - bmo uses this script, don't they?
Yes, and I can't think of anything this patch breaks, so although it worries me,
I think it's probably ok.
(Assignee)

Comment 8

15 years ago
Checking in cvs-update.sh; written
/cvsroot/mozilla/webtools/bugzilla/contrib/cvs-update.sh,v  <--  cvs-update.sh
new revision: 1.2; previous revision: 1.1
done
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.