Closed Bug 1651082 Opened 2 years ago Closed 2 years ago

Solaris configure error: AttributeError: symbols_link_arg


(Firefox Build System :: General, defect)



(firefox80 fixed)

Tracking Status
firefox80 --- fixed


(Reporter: petr.sumbera, Assigned: petr.sumbera, NeedInfo)



(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:77.0) Gecko/20100101 Firefox/77.0

Steps to reproduce:

I see following during build on Solaris:

js/src> creating ./
js/src> Creating config.status
Creating config.status
Reticulating splines...
 0:01.92 File already read. Skipping: /builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/gfx/angle/targets/angle_common/
Traceback (most recent call last):
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/", line 181, in <module>
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/", line 57, in main
    return config_status(config)
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/", line 176, in config_status
    return config_status(args=[], **normalize(sanitized_config))
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/python/mozbuild/mozbuild/", line 148, in config_status
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/python/mozbuild/mozbuild/backend/", line 130, in consume
    if (not self.consume_object(obj) and
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/python/mozbuild/mozbuild/backend/", line 619, in consume_object
    self._process_shared_library(obj, backend_file)
  File "/builds/psumbera/userland-ff-78/components/desktop/firefox/firefox-78.0.1/python/mozbuild/mozbuild/backend/", line 1279, in _process_shared_library
    if libdef.symbols_link_arg:
AttributeError: symbols_link_arg

Following patch fixes the issue:

--- a/python/mozbuild/mozbuild/frontend/ Fri Jun 07 12:49:57 2019 +0200
+++ b/python/mozbuild/mozbuild/frontend/ Wed Sep 25 13:42:06 2019 +0200
@@ -835,6 +835,8 @@
             os_target = context.config.substs['OS_TARGET']
             if os_target == 'Darwin':
                 self.symbols_link_arg = '-Wl,-exported_symbols_list,' + self.symbols_file
+            elif os_target == 'SunOS':
+                self.symbols_link_arg = '-z gnu-version-script-compat -Wl,--version-script,' + self.symbols_file
             elif os_target == 'WINNT':
                 if context.config.substs.get('GNU_CC'):
                     self.symbols_link_arg = self.symbols_file

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: Untriaged → General
Product: Firefox → Firefox Build System

Thanks Petr, do you want to submit that as a Phabricator patch so it can be reviewed and landed? I can do it for you if not.

Flags: needinfo?(petr.sumbera)

I have tried to submit it via Phabricator. But something went wrong and it's not linked to the bug...

Can you please have look what I did wrong and what can be done to fix it?

Flags: needinfo?(petr.sumbera)
Assignee: nobody → petr.sumbera

I think it's fixed now. Do I need to do anything else so it's pushed to the gate?

Looks good, now that you have an approving review you can queue the patch for landing by clicking "View Stack in Lando" on that Phabricator page, then choosing to land.

Lando says "You have insufficient permissions to land. Level 3 Commit Access is required." How can I get scm_level_3?

There used to be 'checkin-needed'. But not now. There is just 'checkin-needed-tb'.

Not in bugzilla, in phabricator. See the link in comment 8.

Pushed by
Add support for Solaris linker r=froydnj
Pushed by
Add support for Solaris linker r=froydnj
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.