Opened 7 weeks ago

Closed 7 weeks ago

#6721 closed change (fixed)

Update buildtools dependency to 9a56d76cd951

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

Description

Background

With #6647, we started to match domains encoded as Punycode (instead of Unicode). The build.py 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 build.py 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 (3)

comment:1 Changed 7 weeks ago by sebastian

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

comment:2 Changed 7 weeks ago by abpbot

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

comment:3 Changed 7 weeks 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
Note: See TracTickets for help on using tickets.