Closed Bug 86651 Opened 23 years ago Closed 22 years ago

contrib/cvs-update.sh sets sticky dates

Categories

(Bugzilla :: Bugzilla-General, defect, P3)

2.13
x86
Linux
defect

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: jacob, Assigned: jacob)

Details

Attachments

(1 file)

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.
Attached patch patchSplinter Review
Keywords: patch
Target Milestone: --- → Bugzilla 2.16
Priority: -- → P3
-> 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 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.
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
Closed: 22 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.

Attachment

General

Created:
Updated:
Size: