Opened on 08/06/2014 at 07:01:54 AM

Closed on 08/06/2014 at 08:49:02 AM

#1176 closed change (fixed)

Don't include the apt module from nginx

Reported by: trev Assignee: trev
Priority: P3 Milestone:
Module: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

​http://codereview.adblockplus.org/5962684784705536

Description

Background

We include the apt module in the nginx module. This should be unnecessary as it is already used by the base module. Supposedly, this also caused issue with an ENC-based setup.

What to change

Don't include the apt module.

Attachments (0)

Change History (2)

comment:1 Changed on 08/06/2014 at 07:03:40 AM by trev

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

comment:2 Changed on 08/06/2014 at 08:49:02 AM by trev

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

Fixed: ​https://hg.adblockplus.org/infrastructure/rev/3f7aa24cdd34

Add Comment

Modify Ticket

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