Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Issue 1279413002: Remove execute bit from binaries install in the toolchain (Closed)

Created:
5 years, 4 months ago by Sam Clegg
Modified:
5 years, 4 months ago
Reviewers:
binji
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/external/naclports.git@master
Target Ref:
refs/heads/master
Project:
naclports
Visibility:
Public.

Description

Remove execute bit from binaries install in the toolchain Previously they were being exluded but the xfonts build process requires the fc-cache nexe to be installed in the toolchain. TBR=binji Committed: https://chromium.googlesource.com/external/naclports/+/80c0fe3242838ba9e72bf24345e11f00595bdc45

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -27 lines) Patch
M lib/naclports/binary_package.py View 5 chunks +17 lines, -23 lines 0 comments Download
M lib/naclports/source_package.py View 1 chunk +1 line, -1 line 0 comments Download
M lib/naclports/tests/test_binary_package.py View 1 chunk +5 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Sam Clegg
Committed patchset #1 (id:1) manually as 80c0fe3242838ba9e72bf24345e11f00595bdc45 (presubmit successful).
5 years, 4 months ago (2015-08-11 05:37:00 UTC) #1
binji
5 years, 4 months ago (2015-08-11 20:45:48 UTC) #2
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698