Created attachment 658699 [details] [diff] [review] Move DirectoryProvider into mail/components/build directory Because shell directory is built only if target platform is some platforms but DirectoryProvider is not platform specific.
Comment on attachment 658699 [details] [diff] [review] Move DirectoryProvider into mail/components/build directory I don't think I'd want this in the build directory. Maybe in a dirprovider dir similar to FF, but I'd really prefer to alter the ifdefs for entering the directory (I think nsSetDefaultMail.js would be installed on all platforms as well for example).
Jcranmer, what do you think about this?
I would concur with Standard8 in the r-. Given that specifying C++ libraries has become rather trivial (FINAL_LIBRARY + SOURCES), and looking at the history of this file, it seems that this ought to be moved to mail/components. Maybe we can even fix it to use JS again instead of C++, since the issue was the xpconnect console error that I'm pretty sure we're now forcibly ignoring.
So would you be able to take this bug?