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

Updated the build.py to be compatible with python3 - #349

Open
opg7371 wants to merge 3 commits into
privly:developfrom
opg7371:develop-python3-build-file
Open

Updated the build.py to be compatible with python3#349
opg7371 wants to merge 3 commits into
privly:developfrom
opg7371:develop-python3-build-file

Conversation

@opg7371

@opg7371 opg7371 commented Feb 21, 2016

Copy link
Copy Markdown

The changes that i made in the build.py for privly applications is made in from line 70 to 72. The problem was that the type of prettyHTML came out to be different in python2(str) and python3(bytes).So i committed the changes can be seen in line number 78 to 86.The lines changed can be seen in the above commit.

@opg7371

opg7371 commented Feb 21, 2016

Copy link
Copy Markdown
Author

This branch has some comments which might help
#313

Comment thread build.py Outdated
prettyHTML)
#The if statement is needed to maintain support for both python2 and python3
if(type(prettyHTML)!=str):
#For python2 the type is String and for python3 the type is bytes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: the whitespace is inconsistent here

@opg7371
opg7371 force-pushed the develop-python3-build-file branch from c1d2305 to c4fca26 Compare February 21, 2016 21:08
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.

2 participants