Closed Bug 1804298 Opened 2 years ago Closed 2 years ago

Make fdlibm vendorable by mach vendor

Categories

(Core :: JavaScript Engine, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
109 Branch
Tracking Status
firefox109 --- fixed

People

(Reporter: tjr, Assigned: tjr)

References

Details

Attachments

(4 files)

We are trying to add an informative moz.yaml file for all third party code in Firefox. Optionally, this file can include (machine-read) instructions for vendoring a new update of the library in. (This typically replaces existing in-tree scripts.)

With those instructions, we can also have Updatebot look for upstream changes and automatically detect them and file a bug, send to try, attach a patch, etc.

In order to make Bug 1760633 much easier, I'd like to make fdlibm vendorable with mach vendor.

You would run ./mach vendor modules/fdlibm/moz.yaml to perform the update. Because fdlibm has patches you'd first run ./mach vendor modules/fdlibm/moz.yaml --patch-mode=none, commit that, and then ./mach vendor modules/fdlibm/moz.yaml --patch-mode=only to apply the patches, and then commit that. If the patches need updating, that command will fail and you can manually change the files and update the patch files so it will work next time.

  • make sure the patch files are sorted before we apply them
  • strip trailing slashes from the repo url before we add more
  • Allow the individual-files action to be used by individual-files

If we want to use individual-files for a lot of files, all
of which are going the same place, it is very verbose and
repetitive. Create an alternate, mutually-exclusive way of
specifying individual-files in a moz.yaml

Depends on D163968

Remove the old update scripts so we can use ./mach vendor
to vendor it.

Depends on D163969

For mach vendor, patches are relative to the directory the
third party code is vendored into.

Depends on D163970

Pushed by tritter@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/16e0b1dd9ea3 mach vendor Fixes/Minor Improvements r=andi https://hg.mozilla.org/integration/autoland/rev/10959074fd6b mach vendor support for an alternate, more terse way of individual-files r=andi https://hg.mozilla.org/integration/autoland/rev/1750fca4189b Add a moz.yaml for fdlibm and make it vendor-able r=arai https://hg.mozilla.org/integration/autoland/rev/abf7ce069748 Update the fdlibm patches to be vendored by mach vendor r=arai
Depends on: 1847898
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: