Skip to content

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.

对齐,看着舒服一点。

# 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解释器

# 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