tools/list_pkg_directories.py - Issue 11444015: Create mock pub links in the build for all packages under pkg, not a hard-coded list

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: tools/list_pkg_directories.py

Issue 11444015: Create mock pub links in the build for all packages under pkg, not a hard-coded list (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/

Patch Set: Created 8 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

OLD	NEW
(Empty)
	1 #!/usr/bin/env python
	ahe 2012/12/06 13:44:39 The style guide we use for Python is http://www.ch The style guide we use for Python is http://www.chromium.org/developers/coding-style. As far as I can tell, this style guide does not require you to wrap scripts like this in: if __name__ == '__main__': sys.exit(main()) So I guess this is fine. Alan Knight 2012/12/06 22:09:23 OK. I don't really know Python, so I don't even kn Show quoted text On 2012/12/06 13:44:39, ahe wrote: > The style guide we use for Python is > http://www.chromium.org/developers/coding-style. > > As far as I can tell, this style guide does not require you to wrap scripts like > this in: > > if __name__ == '__main__': > sys.exit(main()) > > So I guess this is fine. OK. I don't really know Python, so I don't even know what that would do. ahe 2012/12/06 23:02:28 See: http://google-styleguide.googlecode.com/svn/t Show quoted text On 2012/12/06 22:09:23, Alan Knight wrote: > On 2012/12/06 13:44:39, ahe wrote: > > The style guide we use for Python is > > http://www.chromium.org/developers/coding-style. > > > > As far as I can tell, this style guide does not require you to wrap scripts > like > > this in: > > > > if __name__ == '__main__': > > sys.exit(main()) > > > > So I guess this is fine. > > OK. I don't really know Python, so I don't even know what that would do. See: http://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Main#Main
	2 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file

	3 # for details. All rights reserved. Use of this source code is governed by a

	4 # BSD-style license that can be found in the LICENSE file.

	5

	6

	7 '''Tool for listing the directories under pkg, with their lib directories.

	8 Used in pkg.gyp. Lists all of the directories in the current directory

	9 which have a lib subdirectory.

	10

	11 Usage:

	12 python tools/make_links.py TARGET SOURCES...
	ahe 2012/12/06 13:44:39 The comment is out of date. The comment is out of date. Alan Knight 2012/12/06 22:09:23 Done. Show quoted text On 2012/12/06 13:44:39, ahe wrote: > The comment is out of date. Done.
	13 '''

	14 import os

	15

	16 paths = map(lambda x: x + '/lib', filter(os.path.isdir, os.listdir('.')))

	17 filter(lambda x: os.path.exists(x), paths)

	18
	ahe 2012/12/06 13:44:39 Extra line. Extra line. Alan Knight 2012/12/06 22:09:23 Done. Show quoted text On 2012/12/06 13:44:39, ahe wrote: > Extra line. Done.
OLD	NEW

« no previous file with comments | « pkg/pkg.gyp ('k') | no next file » | no next file with comments »