make source-package fails on comm-esr10 because MKDIR is undefined

RESOLVED FIXED

Status

Thunderbird
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: nthomas, Assigned: standard8)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
First ESR release on the new infra has:

make source-package hg-bundle HG_BUNDLE_REVISION=7c765c397d25
 in dir /builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10/objdir-tb
make[1]: Entering directory `/builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10/objdir-tb/mail/installer'
Packaging source tarball...
p ../../mozilla/dist/source/
make[1]: p: Command not found
make[1]: [source-package] Error 127 (ignored)
(cd /builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10 && gtar -c --owner=0 --group=0 --numeric-owner --mode="go-w" --exclude=".hg*" --exclude="CVS" --exclude=".cvs*" --exclude=".mozconfig*" --exclude="*.pyc" --exclude="/builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10/mozilla/Makefile" --exclude="/builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10/mozilla/dist" --exclude="objdir-tb" -f - ../comm-esr10) | bzip2 -vf > ../../mozilla/dist/source/thunderbird-10.0.5esr.source.tar.bz2
/bin/sh: ../../mozilla/dist/source/thunderbird-10.0.5esr.source.tar.bz2: No such file or directory
gtar: Removing leading `../' from member names
make[1]: *** [source-package] Error 1
make[1]: Leaving directory `/builds/slave/tb-rel-comm-esr10-thunderbird-source/comm-esr10/objdir-tb/mail/installer'
make: *** [source-package] Error 2

I think we're getting to
 http://mxr.mozilla.org/comm-esr10/source/mozilla/toolkit/mozapps/installer/packager.mk#995
and mozilla/config/config.mk's definition of MKDIR hasn't been loaded. The FF 10.0.5 ESR build didn't hit this issue so this seems likely to be related to the mail build system.
(Assignee)

Comment 1

6 years ago
Ok, I think I see this, we need bug 727406 landing on esr so that MKDIR is defined - I hadn't twigged that needed to be on esr as well.
Assignee: nobody → mbanner
Depends on: 727406
(Reporter)

Comment 2

6 years ago
Worked in build2.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.