Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#3760 closed change (fixed)

Remove the "opera" build target

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



Historically, we have seperate builds for Chrome and Opera. That was because early Chromuim-based versions of Opera still had quite some differences requiring changes to the build. However, meanwhile our Chrome and Opera builds are identical, except for the browser name mentioned in the short description which can just be changed. So we don't need to support a separate build target for Opera anymore, but can simply provide the same builds as for Chrome.

What to change

  • Merge the short description of Adblock Plus for Chrome, Opera and Safari
  • Remove support for ./ -t opera

Change History (5)

comment:1 Changed 5 years ago by sebastian

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

comment:2 Changed 5 years ago by abpbot

A commit referencing this issue has landed:

comment:3 Changed 5 years ago by abpbot

A commit referencing this issue has landed:

comment:4 Changed 5 years ago by sebastian

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed 5 years ago by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Works as expected. Could build and then install Chrome / Opera builds as described in the README after downloading dependencies.

adblockpluschrome / 37693c923e69

Version 0, edited 5 years ago by Ross (next)
Note: See TracTickets for help on using tickets.