Opened on 09/11/2014 at 06:40:05 PM

Closed on 01/11/2015 at 07:34:36 PM

#1364 closed change (fixed)

Make Element Hiding Helper & Co. use the new dependencies mechanism

Reported by: trev Assignee: trev
Priority: P4 Milestone: Element-Hiding-Helper-1.3.1
Module: Extensions-for-Adblock-Plus Keywords:
Cc: Blocked By: #170, #1363
Blocking: #1588, #1757 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by kzar)


In #170 we introduced our custom dependency mechanism, we should have the simple extensions use it first.

What to change

Change the following extensions:

  • Adblock Plus Crawler
  • Customization for Adblock Plus
  • Diagnostics for Adblock Plus
  • Adblock Plus Tests
  • Element Hiding Helper for Adblock Plus
  • URL Fixer
  • Add the script to the root of each repository.
  • Migrate the contents of the .hgsub and .hgsubstate files to a dependencies file for use by the script. (See ticket #170 and the script itself for more information about the format. See the infrastructure repository for a working example.)
  • Update to execute the script as a subprocess before running tasks.

Attachments (0)

Change History (9)

comment:1 Changed on 09/11/2014 at 07:00:22 PM by trev

  • Owner set to trev

comment:2 Changed on 09/11/2014 at 07:08:21 PM by trev

  • Owner trev deleted

comment:3 Changed on 11/20/2014 at 03:13:58 PM by trev

  • Blocking 1588 added

comment:4 Changed on 12/01/2014 at 03:02:02 PM by poz2k4444

  • Owner set to poz2k4444

comment:5 Changed on 01/06/2015 at 01:23:06 PM by poz2k4444

  • Blocking 1757 added

comment:6 Changed on 01/06/2015 at 04:01:06 PM by poz2k4444

  • Description modified (diff)

comment:7 Changed on 01/09/2015 at 09:56:14 AM by kzar

  • Description modified (diff)

comment:8 Changed on 01/10/2015 at 09:07:38 AM by trev

  • Owner changed from poz2k4444 to trev
  • Review URL(s) modified (diff)

comment:9 Changed on 01/11/2015 at 07:34:36 PM by trev

  • Milestone set to Element-Hiding-Helper-next
  • Resolution set to fixed
  • Status changed from new to closed


I compared the generated builds before and after the change. Crawler, Customization and Diagnostics has rather outdated buildtools dependencies so there were additional changes - all related to buildtools fixes. For the other extensions identical builds with minimal differences (esp. license texts) were generated.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
Note: See TracTickets for help on using tickets.