Closed
Bug 1305912
Opened 9 years ago
Closed 9 years ago
source step busted at |configure|
Categories
(SeaMonkey :: Release Engineering, defect)
SeaMonkey
Release Engineering
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.46
People
(Reporter: ewong, Assigned: ewong)
References
Details
Attachments
(4 files, 2 obsolete files)
|
3.09 KB,
patch
|
philip.chee
:
review+
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-aurora+
iannbugzilla
:
approval-comm-beta+
iannbugzilla
:
approval-comm-release+
|
Details | Diff | Splinter Review |
|
1.22 KB,
patch
|
Details | Diff | Splinter Review | |
|
1.39 KB,
patch
|
ewong
:
review+
philip.chee
:
approval-comm-beta+
philip.chee
:
approval-comm-release+
|
Details | Diff | Splinter Review |
|
1.48 KB,
patch
|
Details | Diff | Splinter Review |
TOOLTOOL_DIR issue? (just filing this bug.. will think on it later)
checking for a shell... /bin/sh
checking for host system type... x86_64-pc-linux-gnu
checking for target system type... x86_64-pc-linux-gnu
checking whether cross compiling... no
checking for pkg_config... not found
DEBUG: pkg_config: Trying /builds/slave/rel-c-rel-source/comm-release/mozilla/gtk3/usr/local/bin/pkg-config
ERROR: Cannot find pkg_config
*** Fix above errors and then restart with\
"make -f client.mk build"
make: *** [configure] Error 1
| Assignee | ||
Comment 1•9 years ago
|
||
oh yes. in light that I remembered somehow that I have some patch
somewhere in some bug that removes the no_tooltool=1 from
the mozconfigs but have forgotten about which bug this belongs
to, I'm going to attach a patch here that removes the no_tooltool=1
from the release mozconfigs.
This means.. build #2. yup.. build 2.
| Assignee | ||
Comment 2•9 years ago
|
||
will do a post-land-review.. or if someone can review this post-haste..
whichever option is the quickest.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #8795600 -
Flags: review?(philip.chee)
Attachment #8795600 -
Flags: review?(iann_bugzilla)
Attachment #8795600 -
Flags: review?(bugspam.Callek)
| Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8795600 [details] [diff] [review]
[mozconfigs] remove no_tooltool=1 [checked-in]
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: none. just can't build in automation
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String changes made by this patch: none
Basically we need this patch so that our release builders
can use the tooltool packages.
Note: the l10n-mozconfig changes were fixed by an earlier patch
so transplanting this patch from c-r to others
will need some finesse.
Attachment #8795600 -
Flags: approval-comm-release?
Attachment #8795600 -
Flags: approval-comm-beta?
Attachment #8795600 -
Flags: approval-comm-aurora?
Comment on attachment 8795600 [details] [diff] [review]
[mozconfigs] remove no_tooltool=1 [checked-in]
r/a=me for all repos
Attachment #8795600 -
Flags: review?(iann_bugzilla)
Attachment #8795600 -
Flags: review+
Attachment #8795600 -
Flags: approval-comm-release?
Attachment #8795600 -
Flags: approval-comm-release+
Attachment #8795600 -
Flags: approval-comm-beta?
Attachment #8795600 -
Flags: approval-comm-beta+
Attachment #8795600 -
Flags: approval-comm-aurora?
Attachment #8795600 -
Flags: approval-comm-aurora+
| Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/releases/comm-release/rev/eb3d525a2f5cf80bae00d00147008548402aa7dc
Bug 1305912 - Remove no_tooltool=1 from release mozconfigs. r+a=IanN
| Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8795616 -
Flags: review?(philip.chee)
| Assignee | ||
Comment 7•9 years ago
|
||
this depends on the "disable compile env during source step"'s mozconfig.
Attachment #8795622 -
Flags: review?(bugspam.Callek)
Updated•9 years ago
|
Attachment #8795600 -
Flags: review?(philip.chee) → review+
Comment 8•9 years ago
|
||
Comment on attachment 8795616 [details] [diff] [review]
[mozconfig] disable compile env during source step
Looks OK
> +export MOZ_DEBUG_SYMBOLS=1
but you might want to add:
ac_add_options --enable-debug-symbols
even if this is a dummy file.
Attachment #8795616 -
Flags: review?(philip.chee) → review+
| Assignee | ||
Comment 9•9 years ago
|
||
Attachment #8795616 -
Attachment is obsolete: true
Attachment #8795628 -
Flags: review+
| Assignee | ||
Comment 10•9 years ago
|
||
Comment on attachment 8795628 [details] [diff] [review]
[mozconfigs] disable compile env for source step.
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: source step in release process doesn't need the compile env, so it's better to disable it.
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String changes made by this patch: none
Attachment #8795628 -
Flags: approval-comm-release?
Attachment #8795628 -
Flags: approval-comm-beta?
Comment 11•9 years ago
|
||
Comment on attachment 8795628 [details] [diff] [review]
[mozconfigs] disable compile env for source step.
a=me for comm-beta comm-release and anything else.
Attachment #8795628 -
Flags: approval-comm-release?
Attachment #8795628 -
Flags: approval-comm-release+
Attachment #8795628 -
Flags: approval-comm-beta?
Attachment #8795628 -
Flags: approval-comm-beta+
| Assignee | ||
Comment 12•9 years ago
|
||
https://hg.mozilla.org/releases/comm-release/rev/762fbf1fa7f3ec770326b73bddc1e6e5d1a415c4
Bug 1305912 - Disable compile environment for source step. r+a=RattyAway
| Assignee | ||
Comment 13•9 years ago
|
||
https://hg.mozilla.org/build/buildbot-configs/rev/77fb9f87b9f175d3e8b845848b1df4720f510148
Bug 1305912 - Use the source mozconfig instead of the release mozconfig for the source step.
| Assignee | ||
Comment 14•9 years ago
|
||
something within the source mozconfig is bugging the source step.
| Assignee | ||
Comment 15•9 years ago
|
||
| Assignee | ||
Comment 16•9 years ago
|
||
https://hg.mozilla.org/releases/comm-release/rev/ead2b3d573c63f5c2b8d68d10b1b0b0f825f981c
Bug 1305912 - Fix for source mozconfig. r+a=bustagefix
| Assignee | ||
Comment 17•9 years ago
|
||
Attachment #8796044 -
Attachment is obsolete: true
Attachment #8796051 -
Flags: review?(bugspam.Callek)
| Assignee | ||
Comment 18•9 years ago
|
||
https://hg.mozilla.org/releases/comm-release/rev/6ada690edc807346f32266afa56054c0e4af9ea5
Bug 1305912 - Fix source mozconfig. r+a=bugfix
| Assignee | ||
Comment 19•9 years ago
|
||
https://hg.mozilla.org/releases/comm-release/rev/013bcbc4360b099e77e0b64ce364cfbc9356d8f8
Bug 1305912 - Remove no_tooltool=1 from release mozconfigs. r+a=IanN
https://hg.mozilla.org/releases/comm-release/rev/55fd7a3517f7836b600e6dec832fd9615894ae05
Bug 1305912 - Disable compile environment for source step. r+a=RattyAway
https://hg.mozilla.org/releases/comm-release/rev/34bd688d21a88f3c13cd8d493d9fd8e494bfae9b
Bug 1305912 - Fix source mozconfig. r+a=bugfix
| Assignee | ||
Updated•9 years ago
|
Attachment #8795600 -
Attachment description: [mozconfigs] remove no_tooltool=1 → [mozconfigs] remove no_tooltool=1 [checked-in]
Attachment #8795600 -
Flags: review?(bugspam.Callek)
| Assignee | ||
Updated•9 years ago
|
Attachment #8796051 -
Flags: review?(bugspam.Callek)
| Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•9 years ago
|
Target Milestone: --- → seamonkey2.46
| Assignee | ||
Updated•7 years ago
|
Attachment #8795622 -
Flags: review?(bugspam.Callek)
You need to log in
before you can comment on or make changes to this bug.
Description
•