Opened on 06/01/2016 at 01:09:37 PM

Closed on 06/06/2016 at 12:59:42 PM

#4098 closed change (fixed)

Get rid of special build setup for development builds

Reported by: trev Assignee: trev
Priority: P3 Milestone:
Module: Sitescripts Keywords:
Cc: sebastian Blocked By: #4097, #4099, #4108, #4109, #4110, #4111, #4112, #4113, #4114
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29345508/

Description

Background

Development builds script currently assumes that all repositories are present locally. Also, it will always use the same copy of buildtools repository for all builds. While this is fast, it also creates consistency issues, particularly because of using the latest revision of buildtools repository. And we don't really want to keep maintaining local versions of all repositories on the build server.

What to change

Check out extension repository from https://hg.adblockplus.org/ rather than copying a local repository. Run ensure_dependencies.py without any special parameters to get all dependencies checked out as well. Add a an optional SpiderMonkey path to sitescripts.ini and pass it to the build via SPIDERMONKEY_BINARY environment variable (see #4097).

Attachments (0)

Change History (14)

comment:1 Changed on 06/01/2016 at 01:09:54 PM by trev

  • Cc sebastian added

comment:2 Changed on 06/01/2016 at 01:14:32 PM by trev

  • Priority changed from Unknown to P3
  • Ready set

comment:3 Changed on 06/01/2016 at 02:26:00 PM by trev

  • Blocked By 4099 added

comment:4 Changed on 06/01/2016 at 02:58:05 PM by trev

  • Review URL(s) modified (diff)

comment:5 Changed on 06/03/2016 at 01:09:18 PM by trev

  • Blocked By 4108 added

comment:6 Changed on 06/03/2016 at 01:19:49 PM by trev

  • Blocked By 4109 added

comment:7 Changed on 06/03/2016 at 01:21:51 PM by trev

  • Blocked By 4110 added

comment:8 Changed on 06/03/2016 at 01:26:01 PM by trev

  • Blocked By 4111 added

comment:9 Changed on 06/03/2016 at 01:28:38 PM by trev

  • Blocked By 4112 added

comment:10 Changed on 06/03/2016 at 01:31:23 PM by trev

  • Blocked By 4113 added

comment:11 Changed on 06/03/2016 at 01:33:28 PM by trev

  • Blocked By 4114 added

comment:12 Changed on 06/03/2016 at 02:43:27 PM by trev

  • Status changed from new to reviewing

comment:13 Changed on 06/06/2016 at 12:58:27 PM by abpbot

A commit referencing this issue has landed:
Issue 4098 - Get rid of special build setup for development builds

comment:14 Changed on 06/06/2016 at 12:59:42 PM by trev

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

Server configuration has been updated, it should generate new development builds in ~20 minutes.

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