346427 - cvs remove calendar/resources/content/dateUtils.js

Status: VERIFIED FIXED

(Calendar :: Internal Components, defect)

Description

[Simon Paquet [:sipaq]]

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.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

-> default

Comment 2

[Simon Paquet [:sipaq]]

mvl, the file has been removed for nearly a year now. Is it okay, if I remove it?

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

File a patch, put it in my queue. Otherwise, i will forgot about your request.

Comment 4

[Simon Paquet [:sipaq]]

(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.

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

Without a patch, there is nothing to put in my queue. And yes, this change does need review.

Comment 6

[Simon Paquet [:sipaq]]

Attachment: Remove the file

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.

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 258831 [details] [diff] [review]
Remove the file

r=mvl

Comment 8

[Simon Paquet [:sipaq]]

1.8 Branch:
Removing dateUtils.js;
/cvsroot/mozilla/calendar/resources/content/Attic/dateUtils.js,v  <--  dateUtils.js
new revision: delete; previous revision: 1.35.6.1
done

Trunk:
Removing dateUtils.js;
/cvsroot/mozilla/calendar/resources/content/dateUtils.js,v  <--  dateUtils.js
new revision: delete; previous revision: 1.36
done