This would require a huge checking to add relevant "ifdef 192" :-( Bug 513709 has already removed some of these vars.

This ports all of autoconf.mk.in, config.mk and rules.mk. Will need to port the rest of configure.in too. This is not meant for (future) "c-1.9.2": needs to branch first!

Those don't sound too bad for me to do them with ifdefs, actually. I guess you just don't want to do it that way?

(In reply to comment #3) As I implied in comment 1, I don't want to check what's needed to keep or not on 1.9.2. But feel free to do it if you want.

After checking in the attachment 454457 [details] [diff] [review] in bug 516758, I'll regenerate this patch.

leaky was added by too old CVS 1.503 in mozilla. However comm-central don't have leaky. This patch will remove boehm leak detector and add leaky memory tool.https://developer.mozilla.org/en/Debugging_memory_leaks#Leaky

I file a new bug 575509 for porting bug 516758. I suggest "Depends on" -516758 +575509.

Update Serge's patch for the latest comm-central. And additional sync to attachment 399009 [details] [diff] [review] in bug 513924. Serge, do you have any comments? If the patch is OK for you, I'll add a flag for review request.

(In reply to comment #8) > Serge, do you have any comments? > If the patch is OK for you, I'll add a flag for review request. Serge has had real life acting up lately, and has been [mostly] awol from mozilla stuff, not sure if waiting on an OK from him first is the best use of time here, but I'll leave the choice up to you for now.

(In reply to comment #9) I call Serge only because the initial patch is generated by him. I have had a plan to request review to you after Serge's reply. If you can review the attachment 462031 [details] [diff] [review], it would be happy.

Comment on attachment 462031 [details] [diff] [review] Sync for comm-central v3 O whops, I bitrotted your rules.mk change in Bug 558518. I'll review this regardless tonight. Whoever gets "done" first (as in landed) I'll be sure to correct the patch for the other bug.

Comment on attachment 462031 [details] [diff] [review] Sync for comm-central v3 (In reply to Bug 513924 comment #17) > Comment on attachment 399009 [details] [diff] [review] > Patch v1.1 > > if test "$MOZ_VIEW_SOURCE"; then > AC_DEFINE(MOZ_VIEW_SOURCE) > fi > > Feels like we could just remove this completely and scrub the codebase. I'm > fine with doing it in a followup, though. With this please *either* remove this from us now, or file a followup for ted. (We can address both c-c and m-c in said followup) [if you remove now, we need it gone from autoconf.mk too] > if test "$MOZ_JSLOADER"; then > AC_DEFINE(MOZ_JSLOADER) > fi > > Same with this. Same. Kill the |AC_SUBST(MOZ_NO_INSPECTOR_APIS)| line while your removing the option from our configure, nothing in our side of the codebase needs it Also remove |XPCOM_USE_LEA| from autoconf.mk.in as well. ---- r+ with those things addressed, please attach a new patch and mark checkin-needed in the keywords. (Sorry I didn't get to this as early as I had hoped)

Self note: Need to file Bugs for GC_LEAK_DETECTOR for directory/c-sdk, and nspr (including mozilla/config/nspr/build.mk)

Comment on attachment 462031 [details] [diff] [review] Sync for comm-central v3 (In reply to comment #6) > leaky was added by too old CVS 1.503 in mozilla. However comm-central don't > have leaky. It doesn't because I removed it in http://hg.mozilla.org/comm-central/rev/97196906ae5d (and I don't see why it would need to be readded.)

(In reply to comment #12) For MOZ_VIEW_SOURCE and MOZ_JSLOADER issue, I removed them from configure.in and config/autoconf.mk.in because definitions are not used in c-c part. I also removed MOZ_NO_INSPECTOR_APIS and XPCOM_USE_LEA from config/autoconf.mk.in and MOZ_NO_INSPECTOR_APIS from configure.in (In reply to comment #14) > It doesn't because I removed it in > http://hg.mozilla.org/comm-central/rev/97196906ae5d > (and I don't see why it would need to be readded.) Sorry. I miss it. I removed MOZ_LEAKY part from my previous patch.

Checked in: http://hg.mozilla.org/comm-central/rev/311ea60cc2b8

(In reply to comment #13) > Self note: Need to file Bugs for GC_LEAK_DETECTOR for directory/c-sdk, and nspr > (including mozilla/config/nspr/build.mk) Bug 589505 and Bug 589504 respectively. (In reply to comment #12) > Comment on attachment 462031 [details] [diff] [review] > Sync for comm-central v3 > > (In reply to Bug 513924 comment #17) > > Comment on attachment 399009 [details] [diff] [review] [details] > > Patch v1.1 > > > > if test "$MOZ_VIEW_SOURCE"; then ... > > Feels like we could just remove this completely and scrub the codebase. I'm > > fine with doing it in a followup, though. Bug 589506 > > if test "$MOZ_JSLOADER"; then ... > > Same with this. Same bug