Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

[WebAPI] Add python packagement files #753

Merged
merged 1 commit into from
Aug 26, 2014
Merged

[WebAPI] Add python packagement files #753

merged 1 commit into from
Aug 26, 2014

Conversation

haoyunfeix
Copy link
Contributor

Signed-off-by: Hao yunfeix yunfeix.hao@intel.com

Signed-off-by: Hao yunfeix <yunfeix.hao@intel.com>
@zqzhang
Copy link
Contributor

zqzhang commented Aug 25, 2014

I got github notification as

  • A webapi/tct-2dtransforms-css3-tests/inst.apk.py (107)
  • A webapi/tct-2dtransforms-css3-tests/inst.wgt.py (207)
  • A webapi/tct-2dtransforms-css3-tests/inst.xpk.py (208)
  • A webapi/tct-2dtransforms-css3-tests/suite.json (54)

Most of the inst.*.py files have the same line of code. Can we redesign these scripts to put one of the copies into tools/build/? Personally, I expect we maintain one or several copies of the scripts together rather than one copy in each test suite.

@yugang @jennycao @cicili @ibelem thoughts?

@zqzhang
Copy link
Contributor

zqzhang commented Aug 25, 2014

Showing 692 changed files with 34,030 additions and 27 deletions.

I hate so many duplicated changes too.

@jennycao
Copy link
Contributor

merge the PR first, We will analysis the solution for better install.py maintain:)

jennycao added a commit that referenced this pull request Aug 26, 2014
…agement-files

[WebAPI] Add python packagement files
@jennycao jennycao merged commit 6b3416b into crosswalk-project:master Aug 26, 2014
@haoyunfeix
Copy link
Contributor Author

Add the UT result to share folder, doc/Neusoft/UT_result/PR753

@haoyunfeix haoyunfeix deleted the yunfei/webapi-add-python-packagement-files branch August 26, 2014 13:57
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.

None yet

3 participants