Webrtc breaks builds with symlinked objdirs

RESOLVED FIXED in mozilla16

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jesup, Assigned: jesup)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment, 1 obsolete attachment)

The assumption is that this is due to use of relative paths to try to get 'out' of the objdir.  Likely the problem is largely in mozmake.py; perhaps in configure.in, media/webrtc/Makefile.in or related spots.
Comment on attachment 640497 [details] [diff] [review]
remove relative topsrcdir/srcdir/etc paths from gyp-sourced Makefiles - fixes symlinked objdirs

This appears to fix all the ../* paths in topsrcdir/srcdir/VPATH in gyp-generated Makefiles.

A grep on the makefiles showed no relative paths left after applying this patch and generating an objdir through a symlink, and it builds.
Attachment #640497 - Flags: review?(ted.mielczarek)
Comment on attachment 640497 [details] [diff] [review]
remove relative topsrcdir/srcdir/etc paths from gyp-sourced Makefiles - fixes symlinked objdirs

Review of attachment 640497 [details] [diff] [review]:
-----------------------------------------------------------------

Fixes the bustage for me.  Thanks Randell!
Attachment #640497 - Flags: feedback+
FWIW: my Linux 64 bit SeaMonkey debug builds are also affected by this bug and this patch does help indeed.
Attachment #640497 - Attachment is obsolete: true
Attachment #640497 - Flags: review?(ted.mielczarek)
Comment on attachment 641185 [details] [diff] [review]
remove relative topsrcdir/srcdir/etc paths from gyp-sourced Makefiles - fixes symlinked objdirs (2)

Uses OS to decide to do absolute paths or not (always relative on windows to make it work with 'make' as well as 'pymake').

I'll fix the missing space in the first change before checkin
Attachment #641185 - Flags: review?(ted.mielczarek)
Comment on attachment 641185 [details] [diff] [review]
remove relative topsrcdir/srcdir/etc paths from gyp-sourced Makefiles - fixes symlinked objdirs (2)

Review of attachment 641185 [details] [diff] [review]:
-----------------------------------------------------------------

::: media/webrtc/trunk/tools/gyp/pylib/gyp/generator/mozmake.py
@@ +197,5 @@
>  def Compilable(filename):
>    return os.path.splitext(filename)[1] in COMPILABLE_EXTENSIONS
>  
>  class MakefileGenerator(object):
> +  def __init__(self, target_dicts, data, options, depth, topsrcdir, relative_topsrcdir, relative_srcdir, output_dir,flavor):

nit: space after the comma
Attachment #641185 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/641d061b9e19
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.