Closed Bug 1350055 Opened 3 years ago Closed 3 years ago

--disable-webrtc build fails with "netwerk/ipc/PNecko.ipdl:27: error: can't locate include file `PStunAddrsRequest.ipdl'"

Categories

(Core :: WebRTC, defect, P1)

55 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox52 --- unaffected
firefox-esr52 --- unaffected
firefox53 --- unaffected
firefox54 --- unaffected
firefox55 --- fixed

People

(Reporter: mats, Assigned: mjf)

References

Details

(Keywords: regression)

Attachments

(1 file)

STEPS TO REPRODUCE
1. add "ac_add_options --disable-webrtc" to your .mozconfig file
2. rebuild with an empty $OBJDIR

ACTUAL RESULTS
[...]
 0:33.29 shistory.xpt
 0:33.30   in file included from `/md1/src/inbound/accessible/ipc/other/PDocAccessible.ipdl', line 7:
 0:33.30   in file included from `/md1/src/inbound/ipc/glue/PFileDescriptorSet.ipdl', line 5:
 0:33.30   in file included from `/md1/src/inbound/ipc/glue/PBackground.ipdl', line 6:
 0:33.30   in file included from `/md1/src/inbound/dom/indexedDB/PBackgroundIDBFactory.ipdl', line 6:
 0:33.30   in file included from `/md1/src/inbound/dom/indexedDB/PBackgroundIDBDatabase.ipdl', line 6:
 0:33.30   in file included from `/md1/src/inbound/dom/indexedDB/PBackgroundIDBDatabaseRequest.ipdl', line 6:
 0:33.30   in file included from `/md1/src/inbound/dom/filehandle/PBackgroundMutableFile.ipdl', line 5:
 0:33.30   in file included from `/md1/src/inbound/dom/filehandle/PBackgroundFileHandle.ipdl', line 5:
 0:33.30   in file included from `/md1/src/inbound/dom/filehandle/PBackgroundFileRequest.ipdl', line 6:
 0:33.30   in file included from `/md1/src/inbound/dom/file/ipc/PBlob.ipdl', line 7:
 0:33.30   in file included from `/md1/src/inbound/dom/ipc/PContent.ipdl', line 14:
 0:33.30   in file included from `/md1/src/inbound/security/manager/ssl/PPSMContentDownloader.ipdl', line 7:
 0:33.30   in file included from `/md1/src/inbound/netwerk/ipc/PChannelDiverter.ipdl', line 7:
 0:33.30   in file included from `/md1/src/inbound/netwerk/protocol/ftp/PFTPChannel.ipdl', line 8:
 0:33.30 /md1/src/inbound/netwerk/ipc/PNecko.ipdl:27: error: can't locate include file `PStunAddrsRequest.ipdl'
 0:33.30 Specification could not be parsed.
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.32 warning: [const] doesn't make sense on builtin types., <class 'xpidl.xpidl.BuiltinLocation'>
 0:33.35 startupcache.xpt
[...]
The PStunAddrsRequest.ipdl file was added in bug 1345511 so I'm guessing
it's a regression from that bug.
Blocks: 1345511
Keywords: regression
Rank: 15
Priority: -- → P1
(In reply to Mats Palmgren (:mats) from comment #1)
> The PStunAddrsRequest.ipdl file was added in bug 1345511 so I'm guessing
> it's a regression from that bug.

Mats, if you could double check that this patch fixes the --disable-webrtc build for you it would be much appreciated.  It builds here on my linux machine now, but I'd love a double check.
Assignee: nobody → mfroman
Flags: needinfo?(mats)
Version: Trunk → 55 Branch
With that patch applied I now get errors when linking libxul.so:
10:28.00 ../../build/unix/gold/ld: error: read-only segment has dynamic relocations
10:28.00 /md1/objdir/md1/src/inbound/dist/include/nsTArray.h:559: error: undefined reference to 'mozilla::NrIceStunAddr::~NrIceStunAddr()'
10:28.00 /md1/objdir/md1/src/inbound/dist/include/nsTArray.h:545: error: undefined reference to 'mozilla::NrIceStunAddr::NrIceStunAddr()'
10:28.00 /md1/objdir/md1/src/inbound/dist/include/mozilla/net/NrIceStunAddrMessageUtils.h:29: error: undefined reference to 'mozilla::NrIceStunAddr::SerializationBufferSize() const'
10:28.00 /md1/objdir/md1/src/inbound/dist/include/mozilla/net/NrIceStunAddrMessageUtils.h:36: error: undefined reference to 'mozilla::NrIceStunAddr::Deserialize(char const*, unsigned long)'
10:28.00 /md1/objdir/md1/src/inbound/dist/include/mozilla/net/NrIceStunAddrMessageUtils.h:18: error: undefined reference to 'mozilla::NrIceStunAddr::SerializationBufferSize() const'
10:28.00 /md1/objdir/md1/src/inbound/dist/include/mozilla/net/NrIceStunAddrMessageUtils.h:20: error: undefined reference to 'mozilla::NrIceStunAddr::Serialize(char*, unsigned long) const'
10:28.00 clang: error: linker command failed with exit code 1 (use -v to see invocation)
Flags: needinfo?(mats)
BTW, my .mozconfig has:
ac_add_options --enable-debug
ac_add_options --disable-optimize
ac_add_options --disable-webrtc
ac_add_options --disable-eme
ac_add_options --disable-accessibility
ac_add_options --disable-crashreporter
ac_add_options --disable-necko-wifi

Can you reproduce with those settings?
Flags: needinfo?(mfroman)
Not with _only_ those changes. :-)  In your log I noticed a clang error and changed my .mozconfig to use clang and I can replicate your failure.
Flags: needinfo?(mfroman)
hg error in cmd: hg identify upstream -r tip:
Mats, could you give this one more try?  It is building with clang and your .mozconfig options on my end.
Flags: needinfo?(mats)
The latest patch works fine for me, thanks!
Flags: needinfo?(mats)
Comment on attachment 8850834 [details]
Bug 1350055 - fix --disable-webrtc build with new PStunAddrsRequest.ipdl

https://reviewboard.mozilla.org/r/123348/#review126472

Looks good to me
Attachment #8850834 - Flags: review?(jduell.mcbugs) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3b7cbfca077a
fix --disable-webrtc build with new PStunAddrsRequest.ipdl r=jduell
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3b7cbfca077a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.