939074 - Infer LIBXUL_LIBRARY from FINAL_LIBRARY

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[Mike Hommey [:glandium]]

Attachment: Infer LIBXUL_LIBRARY from FINAL_LIBRARY

Comment 2

[Mike Hommey [:glandium]]

Attachment: Remove most LIBXUL_LIBRARY

This needs to be folded with the other patch, but that would make review harder. Note that after this there *are* a few LIBXUL_LIBRARY left, that's for gtest, and we can't do otherwise just yet.

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 8333614 [details] [diff] [review]
Infer LIBXUL_LIBRARY from FINAL_LIBRARY

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

::: python/mozbuild/mozbuild/frontend/emitter.py
@@ +113,5 @@
>              libs.values()[0].link_static_lib(reldir, libname)
> +            self._libs[libname][reldir].refcount += 1
> +            # The refcount can't go above 1 right now. It might in the future,
> +            # but that will have to be specifically handled. At which point the
> +            # refcount might have to be a list of referencees, for better error

'references'

@@ +115,5 @@
> +            # The refcount can't go above 1 right now. It might in the future,
> +            # but that will have to be specifically handled. At which point the
> +            # refcount might have to be a list of referencees, for better error
> +            # reporting.
> +            assert self._libs[libname][reldir].refcount <= 1:

Drop the colon here

Comment 4

[Mike Hommey [:glandium]]

(In reply to :Ms2ger from comment #3)
> Comment on attachment 8333614 [details] [diff] [review]
> Infer LIBXUL_LIBRARY from FINAL_LIBRARY
> 
> Review of attachment 8333614 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: python/mozbuild/mozbuild/frontend/emitter.py
> @@ +113,5 @@
> >              libs.values()[0].link_static_lib(reldir, libname)
> > +            self._libs[libname][reldir].refcount += 1
> > +            # The refcount can't go above 1 right now. It might in the future,
> > +            # but that will have to be specifically handled. At which point the
> > +            # refcount might have to be a list of referencees, for better error
> 
> 'references'

That was not a typo. It's probably not a word, though.

> @@ +115,5 @@
> > +            # The refcount can't go above 1 right now. It might in the future,
> > +            # but that will have to be specifically handled. At which point the
> > +            # refcount might have to be a list of referencees, for better error
> > +            # reporting.
> > +            assert self._libs[libname][reldir].refcount <= 1:
> 
> Drop the colon here

Gah, I fixed it but in the wrong patch.

Comment 5

[Mike Hommey [:glandium]]

Attachment: Infer LIBXUL_LIBRARY from FINAL_LIBRARY

Fixed typo.

Comment 6

[Mike Hommey [:glandium]]

Attachment: Remove most LIBXUL_LIBRARY

There was one remaining in mobile/android/components/build.

Comment 7

[Gregory Szorc [:gps]]

Comment on attachment 8333746 [details] [diff] [review]
Infer LIBXUL_LIBRARY from FINAL_LIBRARY

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

Some tests would be nice. I also wouldn't mind some docs in build/docs when this stuff finally gets to a somewhat stable state.

::: python/mozbuild/mozbuild/frontend/emitter.py
@@ +128,5 @@
> +            for path, libdef in libs.items():
> +                # For all root libraries (i.e. libraries that don't have a
> +                # FINAL_LIBRARY), record, for each static library it links
> +                # (recursively), that its FINAL_LIBRARY is that root library.
> +                if libdef.refcount == 0:

if not libdef.refcount:

(I don't like seeing things compared to falsy values, ever)

Comment 8

[Gregory Szorc [:gps]]

Comment on attachment 8333788 [details] [diff] [review]
Remove most LIBXUL_LIBRARY

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

rs=me.

Comment 9

[Mike Hommey [:glandium]]

https://hg.mozilla.org/mozilla-central/rev/6d271ac31d9a
https://hg.mozilla.org/mozilla-central/rev/71cbd5b14aec