two modules named mime

VERIFIED FIXED in mozilla0.9

Status

SeaMonkey
Build Config
P3
normal
VERIFIED FIXED
17 years ago
13 years ago

People

(Reporter: dbaron, Assigned: cls)

Tracking

Trunk
mozilla0.9

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
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.
(Reporter)

Comment 1

17 years ago
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

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


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

Comment 3

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

Comment 4

17 years ago
r=jag.

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

Comment 5

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

Comment 6

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