Opened 2 years ago

Closed 2 years ago

Last modified 22 months ago

#6721 closed change (fixed)

Update buildtools dependency to 9a56d76cd951

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone: Adblock-Plus-3.3-for-Chrome-Opera-Firefox
Module: Platform Keywords:
Cc: Blocked By:
Blocking: #6621 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):



With #6647, we started to match domains encoded as Punycode (instead of Unicode). The updatepsl command has been adapted accordingly in #6706.

In #6621, we want to bundle a different version of the io module dependent on the target platform. Support for this has been added to buildtools with #6625.

Included changes in buildtools

The list of changes imported by this is:

a8b5bbe92c0a Noissue - Adapt best practices for trailing commas Sebastian Noack

What to change

Update the buildtools dependency to:

mercurial git
9a56d76cd951 deb79b0

Hints for testers

  • Make sure that after running updatepsl, there will be no non-ascii charecter in lib/publicSuffixList.js and that internationalized domains are encoded in valid Punycode.
  • Make sure that these changes don't cause any difference in the generated builds.

Change History (4)

comment:1 Changed 2 years ago by sebastian

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 2 years ago by abpbot

A commit referencing this issue has landed:
Issue 6721 - Update buildtools dependency to 9a56d76cd951

comment:3 Changed 2 years ago by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Firefox-next
  • Resolution set to fixed
  • Review URL(s) modified (diff)
  • Status changed from reviewing to closed

comment:4 Changed 22 months ago by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

This looks to be working as expected. Internationalized domains are encoded to Punycode and generated builds work.

Note: See TracTickets for help on using tickets.