Closed Bug 490118 Opened 14 years ago Closed 14 years ago

Drop the redundant /resources/ in mailnews directories


(MailNews Core :: Build Config, defect)

Not set


(Not tracked)

Thunderbird 3.0b4


(Reporter: philor, Assigned: philor)



(1 file, 1 obsolete file)

Attached patch s!/resources/content/!/content/! (obsolete) — Splinter Review
Even though it's cute that my fingers can now type "mailnews/base/prefs/resources/content/accountcreation/" without my needing to think about any part of it, that "resources/" is just pointless. We have exactly one directory named resources that has anything in it other than a single directory named content, and that's just because mime has an abandoned and forgotten pair of .properties files that got copied to locales instead of moved.

A bit harsh on patches, but after bug 390262, everybody's all practiced at tracking down where their files went.
Attachment #374575 - Flags: superreview?(bienvenu)
Attachment #374575 - Flags: review?(neil)
I'm wondering if there's a least bad time to land this, e.g., right around a code freeze. It has minimal risk for run-time, right?
Yep, failing to find a file to jar is a build error (as I discovered when I forgot that mail/ still jars smime from mailnews), so your run-time risks are that I got confused and hit ctrl+k instead of backspacing through resources/ in a line, so we wouldn't be jarring something, or that I screwed up the destination in one of the changes, or that we have something that fails silently for not found and which isn't indexed by MXR and that I failed when I was grepping to back up MXR's opinions.

Or, I guess, that I failed to successfully imagine the reason why we still have those non-localized unjarred copies of localized .properties files, or that :)
Attachment #374575 - Flags: review?(neil) → review+
Attachment #374575 - Flags: superreview?(bienvenu) → superreview+
Whiteboard: [needs a time to land]
Attached patch UnrottedSplinter Review
Sort of regretting the time I spent writing a script to recreate this patch, since (so far) I've only had to unrot it once.
Attachment #374575 - Attachment is obsolete: true
Whiteboard: [needs a time to land] → [needs landing post-freeze]
Checked in:
Closed: 14 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [needs landing post-freeze]
Target Milestone: --- → Thunderbird 3.0b4
You need to log in before you can comment on or make changes to this bug.