get rid of unneeded build system parts in comm-central

RESOLVED FIXED

Status

RESOLVED FIXED
10 years ago
8 years ago

People

(Reporter: kairo, Unassigned)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
The initial import of the comm-central build system done in bug 444686 contains a number of things that mozilla-central needs but comm-central doesn't, esp. wrt configure.in and autoconf.mk.in, probably also config.mk.
We should clean that up so comm-central only has the parts it needs.
(Reporter)

Updated

10 years ago
Depends on: 450964
(Reporter)

Updated

10 years ago
Component: Build Config → Build Config
Product: SeaMonkey → MailNews Core
QA Contact: build-config → build-config
(Reporter)

Comment 1

10 years ago
Created attachment 339519 [details] [diff] [review]
step 1: py- and javaXPCOM

This is a first step patch, removing PyXPCOM and JavaXPCOM stuff left over in comm-central, as well as some necko, mathml, extensions, img-en/decoder and canvas references I forgot to kill when removing those parts already in bug 444686.
There's more to be done, but I more or less fell over those parts when working on the second part of bug 455942 :)
Attachment #339519 - Flags: review?(bugzilla)
I'll review this later, but one thought: If we are removing options from our configure.in (because we don't need them) but they are still able to be passed to/used in core, would it be an idea to add some text to configure --help pointing to the additional possibilities in the mozilla/configure help file?
Attachment #339519 - Flags: review?(bugzilla) → review+
(Reporter)

Comment 3

10 years ago
Comment on attachment 339519 [details] [diff] [review]
step 1: py- and javaXPCOM

Pushed as http://hg.mozilla.org/comm-central/rev/1846693808ff

And Mark, I'm fully with your idea of adding a pointer to mozilla/configure options to configure help, this would be a great addition!
Last changed oct 2008, lets resolve this and use individual bugs for followups.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.