Filing on Shaver's behalf. I'll let him fill in the details as I'm not sure I understand the problem fully.
adding mike to the cc list. he was just fixing something related for OS/2 see #58097 shaver, what's the scoop?
The scoop is (sorry for the delay) that we shouldn't have any compile-time directive like HAVE_MOVEMAIL at all. You shouldn't have to recompile to add or remove movemail support. Aren't mail protocols supposed to be pluggable?
Ok, so can someone explain to me why we have HAVE_MOVEMAIL set in configure.in but also in profile/pref-migrator/src/nsPrefMigration.cpp ? That appears to be the only use of HAVE_MOVEMAIL outside of the mailnews module. If that's the case and HAVE_MOVEMAIL is going to be defined there anyways, can we remove the setting from configure.in and define it in some local mailnews header?
Comment on attachment 50307 [details] [diff] [review] Make -DHAVE_MOVEMAIL localized to mailnews r=jag
Attachment #50307 - Flags: review+
Patch checked in.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
verified all code fixes
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.