Closed Bug 346427 Opened 14 years ago Closed 13 years ago
cvs remove calendar/resources/content/date
dateUtils.js has been removed from the build since April 1st (see bug 322768). I think it is time to remove that file from the cvs repository. Assigning to Joey, since he fixed bug 322768.
Assignee: jminta → nobody
mvl, the file has been removed for nearly a year now. Is it okay, if I remove it?
File a patch, put it in my queue. Otherwise, i will forgot about your request.
(In reply to comment #3) > File a patch, put it in my queue. Otherwise, i will forgot about your request. A real patch is total overkill IMO. If I get a 'Go' from you, I can easily remove it myself, since I have commit privileges.
Without a patch, there is nothing to put in my queue. And yes, this change does need review.
mvl wants a patch, he gets a patch. Please be aware that this "patch" is just a stand-in. The "patch" just removes all the contents of dateUtils.js. If the patch is approved, I will cvs remove the file instead of just deleting all of its contents.
Assignee: nobody → bugzilla
Status: NEW → ASSIGNED
Attachment #258831 - Flags: first-review?(mvl)
Comment on attachment 258831 [details] [diff] [review] Remove the file r=mvl
Attachment #258831 - Flags: first-review?(mvl) → first-review+
1.8 Branch: Removing dateUtils.js; /cvsroot/mozilla/calendar/resources/content/Attic/dateUtils.js,v <-- dateUtils.js new revision: delete; previous revision: 184.108.40.206 done Trunk: Removing dateUtils.js; /cvsroot/mozilla/calendar/resources/content/dateUtils.js,v <-- dateUtils.js new revision: delete; previous revision: 1.36 done
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.