Closed Bug 283156 Opened 20 years ago Closed 20 years ago

make-makefile doesn't like DIRS = ../somewhere-else

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: benjamin, Assigned: shaver)

References

()

Details

Attachments

(2 files)

If you have DIRS = ../somewhere-else, and you remove ../somewhere-else/Makefile,
make-makefiles is supposed to recreate it, but make-makefiles was never taught
about upwards-reaching paths, so it assumes all path-parts are downwards, and
tries to access

$(DEPTH)/../../../somewhere-else
instead of 
$(DEPTH)/../somewhere-else
I'm not able to reproduce this on Linux, though Silver was able to on Windows. 
Silver: can you remind me of your specific perl configuration?
Status: NEW → ASSIGNED
Summary: make-makefiles doesn't like DIRS = ../somewhere-else → make-makefile doesn't like DIRS = ../somewhere-else
Win 2K, MSVC 13.10 (.NET 2003 Toolkit?), ActiveState Perl 5.8.3 IIRC (I'm not at
my box currently). At least I think it was using AS Perl here... I do have
Cygwin's as well.

If you want me to try a debug run of make in this situation (or any other
commands), I can do so when I get back home on Friday (maybe before if I'm
lucky).
Yeah, can you uncomment the |$debug = 1| line in make-makefile, delete
db/sqlite3/src/Makefile, and build in storage/ with the patch from 283057?  I am
not able to reproduce this myself, but my windows environment is not something
I'm tremendously capable in.
I got the same as with the debug commented out when building (it is make that
can't find the file).

For completeness:

M:\tree-main\objdir\storage>make
/usr/bin/make export
make[1]: Entering directory `/cygdrive/m/tree-main/objdir/storage'
make[2]: Entering directory `/cygdrive/m/tree-main/objdir/db/sqlite3/src'
Makefile:93: ../../../../../../../source-HEAD/mozilla/config/rules.mk: No such
file or directory
make[2]: *** No rule to make target
`../../../../../../../source-HEAD/mozilla/config/rules.mk'.  Stop.
make[2]: Leaving directory `/cygdrive/m/tree-main/objdir/db/sqlite3/src'
make[1]: *** [export] Error 2
make[1]: Leaving directory `/cygdrive/m/tree-main/objdir/storage'
make: *** [all] Error 2

Hmm. This time I got something (no idea what I changed to make it do this):

M:\tree-main\objdir\storage>make -sw
make: Entering directory `/cygdrive/m/tree-main/objdir/storage'
make[1]: Entering directory `/cygdrive/m/tree-main/objdir/storage'
ac_given_srcdir = ../../../source-HEAD/mozilla
depth           = ..
ac_given_srcdir = ../../source-HEAD/mozilla
ac_given_srcdir = ../../../source-HEAD/mozilla
depth           = ..
ac_given_srcdir = ../../source-HEAD/mozilla
object_fullpath = /cygdrive/m/tree-main/objdir/storage
object_root     = /cygdrive/m/tree-main/objdir
source_subdir   = storage/
makefiles       = storage/../db/sqlite3/src/Makefile
given_srcdir    = ../../source-HEAD/mozilla
ac_dir     = storage/../db/sqlite3/src
ac_file    = storage/../db/sqlite3/src/Makefile
ac_file_in = ../../source-HEAD/mozilla/storage/../db/sqlite3/src/Makefile.in
srcdir     = ../../../../../../../source-HEAD/mozilla/storage/../db/sqlite3/src
top_srcdir = ../../../../../../../source-HEAD/mozilla
cwd        = /cygdrive/m/tree-main/objdir

creating storage/../db/sqlite3/src/Makefile
Attachment #176182 - Flags: superreview?(benjamin)
Attachment #176182 - Flags: review+
The r+ is from shaver on IRC, for reference.
Attachment #176182 - Flags: superreview?(benjamin) → superreview+
Checked in --> FIXED.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Alas, that doesn't work for all cases (such as those in which ../configure is
invoked, instead of /abspath/to/configure or relpath/to/configure).  I have
something that does, though!  Coming up.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Much better, says my test-builder.
Attachment #176187 - Flags: review?(benjamin)
Comment on attachment 176187 [details] [diff] [review]
Remove both .. and the matching "forward" path components.

Sure. Make sure you don't check in the $debug = 1 change
Attachment #176187 - Flags: review?(benjamin) → review+
Encore avec du FIXED.
Status: REOPENED → RESOLVED
Closed: 20 years ago20 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: