Closed
Bug 86651
Opened 23 years ago
Closed 22 years ago
contrib/cvs-update.sh sets sticky dates
Categories
(Bugzilla :: Bugzilla-General, defect, P3)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: jacob, Assigned: jacob)
Details
Attachments
(1 file)
884 bytes,
patch
|
bbaetz
:
review+
jouni
:
review+
|
Details | Diff | Splinter Review |
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•23 years ago
|
||
Updated•23 years ago
|
Target Milestone: --- → Bugzilla 2.16
Updated•23 years ago
|
Priority: -- → P3
Assignee | ||
Comment 2•23 years ago
|
||
-> New Bugzilla Product
Assignee: tara → jake
Component: Bugzilla → Bugzilla-General
Keywords: review
Product: Webtools → Bugzilla
Version: Bugzilla 2.13 → 2.13
Comment 3•23 years ago
|
||
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 4•22 years ago
|
||
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•22 years ago
|
||
Comment on attachment 39126 [details] [diff] [review] patch r=jouni
Attachment #39126 -
Flags: review+
Comment 6•22 years ago
|
||
myk - bmo uses this script, don't they?
Comment 7•22 years ago
|
||
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•22 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
Closed: 22 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•