Closed
Bug 380650
Opened 18 years ago
Closed 18 years ago
Move passwordmgr/resources/content/* to passwordmgr/content/*
Categories
(Toolkit :: Password Manager, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Dolske, Assigned: Dolske)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
As part of the password manager thwacking, I'd like to have the component use a directory structure like other components. I'm basically replacing everything else under passwordmgr/, except for this one bit.
Existing structure:
-------------------
toolkit/components/passwordmgr/resources/Makefile.in
toolkit/components/passwordmgr/resources/jar.mn
toolkit/components/passwordmgr/resources/content/passwordManager.js
toolkit/components/passwordmgr/resources/content/passwordManager.xul
toolkit/components/passwordmgr/resources/content/passwordManagerCommon.js
toolkit/components/passwordmgr/resources/content/passwordManagerExceptions.js
toolkit/components/passwordmgr/resources/content/passwordManagerExceptions.xul
Move to:
--------
toolkit/components/passwordmgr/content/Makefile.in
toolkit/components/passwordmgr/jar.mn
toolkit/components/passwordmgr/content/passwordManager.js
toolkit/components/passwordmgr/content/passwordManager.xul
toolkit/components/passwordmgr/content/passwordManagerCommon.js
toolkit/components/passwordmgr/content/passwordManagerExceptions.js
toolkit/components/passwordmgr/content/passwordManagerExceptions.xul
A couple of files need tweaked to account for the path change, I'll attach a patch.
Assignee | ||
Comment 1•18 years ago
|
||
This patch doesn't alter files being moved, so it can be applied either before or after the move.
Also, just to be clear, the movement of the above files should be performed as CVS copies to retain the version history.
Updated•18 years ago
|
Attachment #264772 -
Flags: review?(gavin.sharp) → review+
Comment 2•18 years ago
|
||
Checking in allmakefiles.sh;
/cvsroot/mozilla/allmakefiles.sh,v <-- allmakefiles.sh
new revision: 1.662; previous revision: 1.661
done
Checking in toolkit/components/passwordmgr/Makefile.in;
/cvsroot/mozilla/toolkit/components/passwordmgr/Makefile.in,v <-- Makefile.in
new revision: 1.5; previous revision: 1.4
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•18 years ago
|
||
Reopening for a bit. Still need to remove the old directory tree [/toolkit/components/passwordmgr/resources], but will wait a bit to do that since other changes are going on. Less to back out in case of disaster. :-) Forgot to mention that to Colin.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 4•18 years ago
|
||
I checked in the removal of these old files, because I needed to convince one of the unit tests machine to build again.
Status: REOPENED → RESOLVED
Closed: 18 years ago → 18 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•