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

Issue 501086: Adds build infrastracture for Gtk theme. (Closed)

Created:
11 years ago by sky
Modified:
9 years, 7 months ago
Reviewers:
Randall Spangler
CC:
chromium-os-reviews_googlegroups.com
Visibility:
Public.

Description

Adds build infrastracture for Gtk theme.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Updated patch #

Patch Set 3 : Remvoes unneeded import #

Unified diffs Side-by-side diffs Delta from patch set Stats (+278 lines, -5 lines) Patch
M src/package_repo/package-list-prod.txt View 1 chunk +1 line, -0 lines 0 comments Download
M src/platform/theme/SConstruct View 1 1 chunk +0 lines, -3 lines 0 comments Download
A src/platform/theme/debian/changelog View 1 chunk +5 lines, -0 lines 0 comments Download
A src/platform/theme/debian/compat View 1 chunk +1 line, -0 lines 0 comments Download
A src/platform/theme/debian/control View 1 chunk +16 lines, -0 lines 0 comments Download
A src/platform/theme/debian/dirs View 1 chunk +7 lines, -0 lines 0 comments Download
A src/platform/theme/debian/postinst View 1 chunk +44 lines, -0 lines 0 comments Download
A src/platform/theme/debian/postrm View 1 chunk +42 lines, -0 lines 0 comments Download
A src/platform/theme/debian/preinst View 1 chunk +40 lines, -0 lines 0 comments Download
A src/platform/theme/debian/prerm View 1 chunk +43 lines, -0 lines 0 comments Download
A src/platform/theme/debian/rules View 1 chunk +62 lines, -0 lines 0 comments Download
A src/platform/theme/make_pkg.sh View 1 chunk +15 lines, -0 lines 0 comments Download
D src/rootfs_static_data/common/etc/gtk-2.0/gtkrc View 1 chunk +0 lines, -1 line 0 comments Download
M src/scripts/build_platform_packages.sh View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
sky
11 years ago (2009-12-17 20:11:57 UTC) #1
sky
Hang off a second on this. I seemed to have screwed something.
11 years ago (2009-12-17 20:13:56 UTC) #2
Randall Spangler
11 years ago (2009-12-17 21:31:55 UTC) #3
lgtm with one suggestion

http://codereview.chromium.org/501086/diff/1/3
File src/platform/theme/SConstruct (right):

http://codereview.chromium.org/501086/diff/1/3#newcode5
src/platform/theme/SConstruct:5: import os
Should be able to remove these imports.

Powered by Google App Engine
This is Rietveld 408576698