Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

Package with distutils.core#2

Merged
alusco merged 3 commits intogoogle:masterfrom
cambridgehackers:distutils
Oct 30, 2014
Merged

Package with distutils.core#2
alusco merged 3 commits intogoogle:masterfrom
cambridgehackers:distutils

Conversation

@jameyhicks
Copy link
Copy Markdown
Contributor

This is a first cut at packaging python-adb with distutils setup.

Moved source for module "adb" to directory "adb".
Added adb/init.py
Updated and ran adb_test.py and fastboot_test.py.
Added setup.py to build and install module.

Moved source for module "adb" to directory "adb".
Added adb/__init__.py
Updated and ran adb_test.py and fastboot_test.py.
Added setup.py to build and install module.
Comment thread adb_test.py
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Google style is probably going to ask you to use import adb instead of using from imports.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, adb_commands is a module, so this is fine. Google style simply says to not import things inside a module, but to import the module itself (for refactoring's sake).

@alusco
Copy link
Copy Markdown
Contributor

alusco commented Oct 30, 2014

Awesome thanks!

alusco added a commit that referenced this pull request Oct 30, 2014
@alusco alusco merged commit 89dd297 into google:master Oct 30, 2014
@jameyhicks jameyhicks deleted the distutils branch October 30, 2014 12:39
maruel added a commit to maruel/python-adb that referenced this pull request Jun 17, 2016
Mimic self.Shell fallback from SetCPUScalingGovernor in SetCPUSpeed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants