68381 - two modules named mime

Status: VERIFIED FIXED

(SeaMonkey :: Build Config, defect, P3)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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/

Comment 2

[cls]

Switching netwerk/mime to use mimetype works for me.

Comment 3

[cls]

Attachment: Set MODULE=mimetype for netwerk/mime/ and update REQUIRES

Comment 4

[Peter "jag" Annema]

r=jag.

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

Comment 5

[cls]

Patch checked in.  Marking fixed.

Comment 6

[Jon Granrose]

verified.