The default bug view has changed. See this FAQ.
Bug 546484 (NoC192SM)

Error out on building SeaMonkey with 1.9.2 from comm-central

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Build Config
--
major
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: sgautherie)

Tracking

Trunk
seamonkey2.1a1
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
As we decided to unsupport 1.9.2 for SeaMonkey on comm-central, we should error out on building with 1.9.2 from that tree, so people do not end up with broken, unsupported configurations inadvertently.
(Reporter)

Updated

7 years ago
Blocks: 546487
(Assignee)

Comment 1

7 years ago
Created attachment 427258 [details] [diff] [review]
(Av1) Just error out
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #427258 - Flags: review?(kairo)
(Reporter)

Comment 2

7 years ago
Comment on attachment 427258 [details] [diff] [review]
(Av1) Just error out

> # Now is a good time to test for logic errors, define mismatches, etc.
> case "$MOZ_BUILD_APP" in
> xulrunner)
>   if test "$LIBXUL_SDK"; then
>     AC_MSG_ERROR([Building XULRunner --with-libxul-sdk doesn't make sense; XULRunner provides the libxul SDK.])
>   fi
>   ;;
>-esac

Don't join those case statements, they are for different reasons, as the comments explain. I wonder if building XULRunner from comm-central makes any sense at all, actually, but that's a different bug.

> # Special cases where we need to AC_DEFINE something. Also a holdover for apps
> # that haven't made a confvars.sh yet. Don't add new stuff here, use
> # MOZ_BUILD_APP.
>-case "$MOZ_BUILD_APP" in
> suite)
>+  if test "$MOZILLA_1_9_2_BRANCH" = "1"; then
>+    AC_MSG_ERROR([Building SeaMonkey with mozilla1.9.2 is not supported.])
>+  fi
>   AC_DEFINE(MOZ_SUITE)
>   ;;
>-
> mail)
>   AC_DEFINE(MOZ_THUNDERBIRD)
>   ;;
>-
> calendar)
>   AC_DEFINE(MOZ_SUNBIRD)
>   ;;
>-
> esac

Please leave those newlines in.

Also, did you try if we can do this in confvars.sh? If it works, I'd prefer that. The whole block of AC_DEFINEs here should actually go away at some stage, if possible...
(Assignee)

Updated

7 years ago
Blocks: 541723
(Assignee)

Updated

7 years ago
Blocks: 545534
(Assignee)

Comment 3

7 years ago
Created attachment 427354 [details] [diff] [review]
(Av2) Just error out
[Checkin: Comment 5]

Av1, with comment 2 suggestion(s).

(In reply to comment #2)

> I wonder if building XULRunner from comm-central makes any
> sense at all, actually, but that's a different bug.

Yeah, I thought about it too in the meantime... ;-)

> Also, did you try if we can do this in confvars.sh?

configure executes |. "${srcdir}/${MOZ_BUILD_APP}/confvars.sh"|.
I have no idea how to error out (from) there.

> The whole block of AC_DEFINEs here should actually go away at some stage,

Does it really matter for this ("short lived") code?
Attachment #427258 - Attachment is obsolete: true
Attachment #427354 - Flags: review?(kairo)
Attachment #427258 - Flags: review?(kairo)
(Assignee)

Updated

7 years ago
Alias: NoC192SM
Severity: normal → major
Flags: in-testsuite-
(Assignee)

Updated

7 years ago
Blocks: 511884
(Assignee)

Updated

7 years ago
Blocks: 536374
(Assignee)

Updated

7 years ago
Blocks: 540380
(Reporter)

Comment 4

7 years ago
Comment on attachment 427354 [details] [diff] [review]
(Av2) Just error out
[Checkin: Comment 5]

Yes, I don't completely love it, but let's just do it.
Attachment #427354 - Flags: review?(kairo) → review+
(Assignee)

Comment 5

7 years ago
Comment on attachment 427354 [details] [diff] [review]
(Av2) Just error out
[Checkin: Comment 5]


http://hg.mozilla.org/comm-central/rev/0cdce605ea0c
Attachment #427354 - Attachment description: (Av2) Just error out → (Av2) Just error out [Checkin: Comment 5]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Blocks: 540470
(Assignee)

Updated

7 years ago
Blocks: 521624
(Assignee)

Updated

7 years ago
Blocks: 511896
(Assignee)

Updated

7 years ago
Blocks: 547375
(Assignee)

Updated

7 years ago
Blocks: 538753
(Assignee)

Updated

7 years ago
Blocks: 536567
(Assignee)

Updated

7 years ago
No longer blocks: 536567
(Assignee)

Updated

7 years ago
Blocks: 506493
(Assignee)

Updated

7 years ago
Blocks: 547039
(Assignee)

Updated

7 years ago
Blocks: 522712
(Assignee)

Updated

7 years ago
Blocks: 549293
No longer blocks: 540380
You need to log in before you can comment on or make changes to this bug.