17.06 KB, patch
|Details | Diff | Splinter Review|
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/
Switching netwerk/mime to use mimetype works for me.
Created attachment 28775 [details] [diff] [review] Set MODULE=mimetype for netwerk/mime/ and update REQUIRES
r=jag. Remember to get someone to rm -r dist/include/mime on senna...
Patch checked in. Marking fixed.