Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2968 closed change (fixed)

Implement multi inheritance for metadata files

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone:
Module: Automation Keywords:
Cc: sebastian Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29325410

Description

Background

To avoid duplication in forks of Adblock Plus for multiple platforms we need a way to inherit from multiple metadata files. So a "fork" can derive from from metadata.chrome or metadata.safari as well as from their own metadata mixin for example.

What to change

A metadatra file that uses following option, should inherit all options from both metadata.chrome and metadata.custom, while similar to multi inheritance in Python, options from the former file should override options from the latter in case of a conflict.

[default]
inherit = metadata.chrome metadata.custom

Change History (3)

comment:1 Changed 4 years ago by sebastian

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

comment:2 Changed 4 years ago by sebastian

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:3 Changed 4 years ago by sebastian

  • Cc sebastian added

302 is implemented inconsistently across browsers, and seems to be superseded by 303 and 307 in HTTP 1.1. Also see the discussion in that review.

Note: See TracTickets for help on using tickets.