Closed
Bug 490118
Opened 13 years ago
Closed 13 years ago
Drop the redundant /resources/ in mailnews directories
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b4
People
(Reporter: philor, Assigned: philor)
Details
Attachments
(1 file, 1 obsolete file)
93.91 KB,
patch
|
Details | Diff | 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)
Comment 1•13 years ago
|
||
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?
Assignee | ||
Comment 2•13 years ago
|
||
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 jar.mn line, so we wouldn't be jarring something, or that I screwed up the destination in one of the Makefile.in 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 Makefile.win :)
Updated•13 years ago
|
Attachment #374575 -
Flags: review?(neil) → review+
Updated•13 years ago
|
Attachment #374575 -
Flags: superreview?(bienvenu) → superreview+
Assignee | ||
Updated•13 years ago
|
Whiteboard: [needs a time to land]
Assignee | ||
Comment 3•13 years ago
|
||
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
Updated•13 years ago
|
Whiteboard: [needs a time to land] → [needs landing post-freeze]
Comment 4•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/0c8752d8fd17
Status: ASSIGNED → RESOLVED
Closed: 13 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.
Description
•