Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: check_curl.py

Issue 10252013: Implemented curl support for web requests (Closed)
Patch Set: Addressed review comments Created April 12, 2013, 1:37 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | include/AdblockPlus/WebRequest.h » ('j') | include/AdblockPlus/WebRequest.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: check_curl.py
===================================================================
--- a/check_curl.py
+++ b/check_curl.py
@@ -1,16 +1,19 @@
#!/usr/bin/env python
# coding: utf-8
import os, sys, subprocess, tempfile;
baseDir = os.path.abspath(os.path.dirname(__file__))
def check_curl():
- (fd, name) = tempfile.mkstemp(dir=os.path.join(baseDir, 'build'), suffix='.h')
+ buildDir = os.path.join(baseDir, 'build')
+ if not os.path.exists(buildDir):
+ os.makedirs(buildDir);
+ (fd, name) = tempfile.mkstemp(dir=buildDir, suffix='.h')
try:
handle = os.fdopen(fd, 'wb');
handle.write('#include <curl/curl.h>')
handle.close();
# This command won't work for Windows or Android build environments but we
# don't want to use curl there anyway
command = ['/usr/bin/env', 'gcc', '-E', name];
« no previous file with comments | « no previous file | include/AdblockPlus/WebRequest.h » ('j') | include/AdblockPlus/WebRequest.h » ('J')

Powered by Google App Engine
This is Rietveld