two modules named mime

VERIFIED FIXED in mozilla0.9

Status

P3
normal
VERIFIED FIXED
18 years ago
14 years ago

People

(Reporter: dbaron, Assigned: cls)

Tracking

Trunk
mozilla0.9

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

There are two different pieces of code that use MODULE=mime.  This left me
thinking for a bit that dougt was introducing dependencies on mailnews's MIME
code from outside of mailnews.  One or both of these modules should probably be
renamed to avoid confusion.  See:

http://lxr.mozilla.org/seamonkey/search?string=MODULE%09%09%3D%20mime

which shows both netwerk/mime and mailnews/mime.
Er, wait, the netwerk one already uses mimetype in its src directory, perhaps we
should change that in the public directory too (and change the REQUIRES in at least 

netwerk/protocol/res/src/
layout/xml/document/src/
rdf/content/src/
(Assignee)

Comment 2

18 years ago
Switching netwerk/mime to use mimetype works for me.


Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9
(Assignee)

Comment 3

18 years ago
Created attachment 28775 [details] [diff] [review]
Set MODULE=mimetype for netwerk/mime/ and update REQUIRES

Comment 4

18 years ago
r=jag.

Remember to get someone to rm -r dist/include/mime on senna...
(Assignee)

Comment 5

18 years ago
Patch checked in.  Marking fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 6

18 years ago
verified.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.