Last Comment Bug 661403 - configure.in: fix MOZ_INSTALLER code w.r.t. confvars.sh
: configure.in: fix MOZ_INSTALLER code w.r.t. confvars.sh
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla10
Assigned To: Serge Gautherie (:sgautherie)
:
: Gregory Szorc [:gps]
Mentors:
http://mxr.mozilla.org/mozilla-centra...
Depends on: 557824 689370
Blocks: 650322 690369
  Show dependency treegraph
 
Reported: 2011-06-01 15:24 PDT by Serge Gautherie (:sgautherie)
Modified: 2011-09-30 00:19 PDT (History)
2 users (show)
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
(Av1) Remove a redundant and misplaced MOZ_INSTALLER=1 block [Checked in: Comment 3] (994 bytes, patch)
2011-09-26 17:45 PDT, Serge Gautherie (:sgautherie)
ted: review+
Details | Diff | Splinter Review

Description Serge Gautherie (:sgautherie) 2011-06-01 15:24:54 PDT
{
case "$target_os" in
    aix*|solaris*|linux*|mingw*|os2*)
        MOZ_INSTALLER=1
        ;;
esac
}
should be before confvars.sh call, not after.
Comment 1 Serge Gautherie (:sgautherie) 2011-09-26 17:45:31 PDT
Created attachment 562609 [details] [diff] [review]
(Av1) Remove a redundant and misplaced MOZ_INSTALLER=1 block
[Checked in: Comment 3]

I was going to move this block up,
but bug 557824 made MOZ_INSTALLER=1 as default,
then this block can just be removed.
(That bug should have done this.)
Comment 2 Daniel Holbert [:dholbert] 2011-09-29 12:32:05 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/547933c90c5f
Comment 3 Matt Brubeck (:mbrubeck) 2011-09-29 20:44:15 PDT
https://hg.mozilla.org/mozilla-central/rev/547933c90c5f

Note You need to log in before you can comment on or make changes to this bug.