Opened on 11/07/2014 at 01:04:52 PM

Closed on 07/04/2017 at 02:54:51 PM

#1548 closed change (worksforme)

Update gyp to the recent version

Reported by: sergz Assignee:
Priority: Unknown Milestone:
Module: Libadblockplus Keywords:
Cc: fhd Blocked By:
Blocking: #1197 Platform: Unknown
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by sergz)

Background

Current version of gyp does not allow to generate solution/project files for MS Visual Studio 2013.

Note

  • with new gyp there is a trouble. The generated project contains in the custom action "--random-seed" "..\..\..\..\..\..\third_party\v8\tools\gyp\314159265" instead of "--random-seed" "314159265" (the relative path should not be added). Its source is v8_random_seed in libadblockplus\third_party\v8\tools\gyp\v8.gyp.
  • there are our changes in gyp which should be rebased (see https://github.com/adblockplus/gyp/commit/8035ac91ddef78ed6fae19fca67702934c5322a3, it's in fix-issue-339 branch)

Attachments (0)

Change History (8)

comment:1 Changed on 11/13/2015 at 02:08:55 PM by sergz

  • Cc fhd added
  • Description modified (diff)
  • Tester set to Unknown

comment:2 Changed on 11/13/2015 at 04:02:26 PM by fhd

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

Updated. I had to push the entire tree as a new head to hg.adblockplus.org/gyp, because the ID mapping has changed since that clone was first created. By keeping the old head, the old refs still work.

comment:3 Changed on 11/13/2015 at 04:21:54 PM by fhd

  • Resolution fixed deleted
  • Status changed from closed to reopened

We still need Wladimir's old patch to keep the Android build on working. I'll have to merge all changes since then into that branch to keep the old refs working.

comment:4 Changed on 11/14/2015 at 01:47:51 AM by fhd

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

Merged master into fix-issue-339.

comment:5 Changed on 01/21/2016 at 09:43:34 PM by sergz

  • Resolution fixed deleted
  • Status changed from closed to reopened

It's still relevant because our fix in gyp does not help here.

comment:6 Changed on 01/26/2016 at 06:06:26 PM by eric@adblockplus.org

Issue description says "current version of gyp". What version of gyp is that? Which one are we updating to?

comment:7 Changed on 07/04/2017 at 02:54:43 PM by sergz

Gyp is updated to https://github.com/adblockplus/gyp/commit/8ffe7e7b1b82890a7e9ea21176bf039b6c53167b (which is based on 70fa8bbeb2921c51b34598d3eda8e5fc485e952c, 2 years ago (2015-11-12 23:09:31), and includes our changes) as a part of #4907. Although this version is not the recent one, the development of gyp is not active and it seems there is nothing very essential for us there.

So, current version of gyp allows to generate solutions for VS 2013 (what this issue is about, but this version is not relevant anymore) as well as for VS 2015 (what is actually used and it even works with VS 2017). Therefore this issue can be closed, please create a new one when a required functionality appears in gyp repository and there is a demand in it.

comment:8 Changed on 07/04/2017 at 02:54:51 PM by sergz

  • Resolution set to worksforme
  • Status changed from reopened to closed

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