Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check Python if system already equipped one #1092

Merged
merged 4 commits into from
Jan 8, 2017

Conversation

gangliao
Copy link
Contributor

@gangliao gangliao commented Jan 8, 2017

默认先检查系统是否有python, 有的话,就用系统的python.

option(ON_TRAVIS "Running test on travis-ci or not." OFF)
option(ON_COVERALLS "Generating code coverage data on coveralls or not." OFF)
option(COVERALLS_UPLOAD "Uploading the generated coveralls json." ON)
option(WITH_GPU "Compile PaddlePaddle with NVIDIA GPU" ${CUDA_FOUND})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

对齐,看着舒服一点。

@@ -12,6 +12,10 @@
# See the License for the specific language governing permissions and
# limitations under the License.

if(NOT WITH_PYTHON)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Paddle可以不内嵌python解释器

@@ -13,192 +13,210 @@
# limitations under the License.

INCLUDE(ExternalProject)
INCLUDE(python_module)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

加了个判断 如果系统有python,就用系统的python, 不需要下载python

Copy link
Contributor

@tianbingsz tianbingsz left a comment

Choose a reason for hiding this comment

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

Very Good. Thanks.

@gangliao gangliao merged commit c016758 into PaddlePaddle:develop Jan 8, 2017
@gangliao gangliao deleted the python_default branch March 22, 2017 03:01
zhhsplendid pushed a commit to zhhsplendid/Paddle that referenced this pull request Sep 25, 2019
The building and installing documentation should be updated because that the code of inference lib has changed. The current bug is that most of files under the inference library path are missed.
lizexu123 pushed a commit to lizexu123/Paddle that referenced this pull request Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants