summaryrefslogtreecommitdiffstats
path: root/Source/cmFileCommand.cxx
Commit message (Collapse)AuthorAgeFilesLines
* file: Avoid runaway DOWNLOAD/UPLOAD progress reports (#14807)Brad King2014-03-171-0/+5
| | | | | | | | Curl makes progress callbacks frequently but we round to the nearest percent and report only when that changes so that we make at most 101 progress reports. However, when unexpected data beyond the total are transferred the progress can get beyond 100% and lead to unlimited reports. Avoid this case by capping the reported progress to 100%.
* Remove some c_str() calls.Stephen Kelly2014-03-111-71/+71
| | | | | | Use the clang RemoveCStrCalls tool to automatically migrate the code. This was only run on linux, so does not have any positive or negative effect on other platforms.
* file DOWNLOAD: Display the curl result status when a hash mismatch occurs.Matt McCormick2014-01-161-0/+2
|
* Use cmsys::[io]fstream instead of cmsys_ios::[io]fstream.Clinton Stimpson2014-01-071-8/+9
| | | | | Also use SystemTools::Fopen() instead of fopen(). This is to eventually support utf-8 filenames.
* Remove some uses of obsolete 'register' storage specifierSean McBride2013-06-281-2/+2
| | | | Remove the keyword from all Source/* files outside of KWSys.
* Merge topic 'genex-generate-file'Brad King2013-05-281-0/+78
|\ | | | | | | | | b983a58 file: Add GENERATE command to produce files at generate time
| * file: Add GENERATE command to produce files at generate timeStephen Kelly2013-05-241-0/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The idea is to write to a temp file which contains generator expressions, and at generate time, evaluate the generator expressions, and write the result to a file. Because executables on Windows are limited in the length of command line it is possible to use, it is common to write command line arguments to a file instead and specify the file as a source of arguments. This new FILE(GENERATE) subcommand allows the use of generator expressions to create such files so that they can be used with add_custom_command for example.
* | Fix spelling and typos (affecting binary data / module messages)Andreas Mohr2013-05-071-2/+2
|/
* file: Do not remove symlinked directories recursively (#10538)Brad King2013-02-131-1/+2
| | | | | If a symlink points at a directory the symlink should be removed but not the content of the directory.
* CMake: Fix dashboard warningsDavid Cole2012-12-051-1/+1
| | | | ...in the new file and string TIMESTAMP sub-commands
* CMake: Add TIMESTAMP subcommand to string and file commandsNils Gladitz2012-12-051-0/+57
|
* file: remove dead codeAmine Chadly2012-11-021-10/+5
| | | | | | The file command requires at least two arguments, so guarding the GLOB and MAKE_DIRECTORY command is not necessary. Changed it for an assert to keep the protection.
* Merge topic 'file-DOWNLOAD-EXPECTED_HASH'David Cole2012-09-251-17/+18
|\ | | | | | | | | 95a0011 file(DOWNLOAD): Change EXPECTED_HASH to take ALGO=value
| * file(DOWNLOAD): Change EXPECTED_HASH to take ALGO=valueBrad King2012-09-191-17/+18
| | | | | | | | | | | | Make the EXPECTED_HASH option take only a single value instead of two to avoid handling sub-keyword arguments. This is also consistent with URL_HASH in ExternalProject.
* | Merge topic 'file-DOWNLOAD-user-agent'Brad King2012-09-191-0/+3
|\ \ | |/ |/| | | | | 14aff4d file(DOWNLOAD): Add HTTP User-Agent string
| * file(DOWNLOAD): Add HTTP User-Agent stringBrad King2012-09-181-0/+3
| | | | | | | | | | | | | | Some servers require a User-Agent string. The curl command-line tool just sends "curl/$curlver", so do the same. Suggested-by: Fredrik Ehnbom <fehnbom@nvidia.com>
* | file(DOWNLOAD): Make TLS options behave as documentedBrad King2012-09-171-33/+13
| | | | | | | | | | | | | | | | | | The logic added in commit e1c89f08 (file(DOWNLOAD): Add options for SSL, 2012-08-21) did not actually provide the documented behavior. Simplify the implementation to read the variable values first and then replace them with the explicit argument values if encountered. Always set the curl option CURLOPT_SSL_VERIFYPEER to either on or off explicitly instead of depending on the curl default behavior.
* | Rename SSL terminology to TLSBrad King2012-09-141-17/+17
|/ | | | | | TLS has superseded SSL so rename the recently added file(DOWNLOAD) and ExternalProject options using the newer terminology. Drop "CURLOPT" from names because curl is an implementation detail.
* file(DOWNLOAD): Add options for SSLBill Hoffman2012-09-111-0/+67
| | | | | | | Add the ability to request that downloads disable or enable Certificate Authority checking with https ssl downloads. When the option to verify the servers CA is disabled, one may verify download contents with SHA hashes.
* file(DOWNLOAD): Generalize EXPECTED_MD5 to EXPECTED_HASHBill Hoffman2012-09-111-36/+53
| | | | Add support for SHA algorithms.
* Remove trailing whitespace from most CMake and C/C++ codeKitware Robot2012-08-131-5/+5
| | | | | | | | | | | | | | | | | Our Git commit hooks disallow modification or addition of lines with trailing whitespace. Wipe out all remnants of trailing whitespace everywhere except third-party code. Run the following shell code: git ls-files -z -- \ bootstrap doxygen.config '*.readme' \ '*.c' '*.cmake' '*.cpp' '*.cxx' \ '*.el' '*.f' '*.f90' '*.h' '*.in' '*.in.l' '*.java' \ '*.mm' '*.pike' '*.py' '*.txt' '*.vim' | egrep -z -v '^(Utilities/cm|Source/(kwsys|CursesDialog/form)/)' | egrep -z -v '^(Modules/CPack\..*\.in)' | xargs -0 sed -i 's/ \+$//'
* Merge topic 'install-case-sensitive-mac'David Cole2012-06-141-2/+4
|\ | | | | | | | | a41557a install: Fix FILES_MATCHING on case-sensitive Mac filesystems (#13177)
| * install: Fix FILES_MATCHING on case-sensitive Mac filesystems (#13177)Brad King2012-06-131-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Windows and Apple machines have predominantly used case-insensitive filesystems so our file(INSTALL) command uses case-insensitive pattern matching. It is implemented by converting the pattern and file path to lower case before matching. The FILES_MATCHING option is implemented by excluding a path that does not match any pattern unless it is a directory that must be searched recursively. However, the test that an excluded path is a directory is executed on the lower-case path and therefore fails on mixed-case input paths on case-sensitive filesystems. Fix the file(INSTALL) implementation to use the lower-case path only for pattern matching and preserve the original path for tests against the real filesystem.
* | Workaround IBM XL v6 streams seekg bug (#13149)Jim Hague2012-04-191-1/+1
|/ | | | | | It seems that file.seekg(0) will, in some circumstances, cause the next file.getline() to omit the first character it reads. Workaround the bug by seeking from ios::beg explicitly.
* Factor cmInstallType out of cmTarget::TargetTypeBrad King2012-02-271-16/+17
| | | | | | The purpose of the TargetType enumeration was overloaded for install type because install rules were once recorded as targets. Factor the install types out into their own enumeration.
* Disable file() and string() hash commands during bootstrapBrad King2011-11-161-0/+7
| | | | | We do not compile support for the cryptographic hashes during bootstrap. Disable the APIs that use them.
* cmCryptoHash: Provide factory "New" methodBrad King2011-11-161-13/+1
| | | | | Construct a cmCryptoHash subclass instance based on the name of the desired hash algorithm.
* Add file(SHA*) commands to compute cryptographic hashesBrad King2011-11-161-1/+16
| | | | Add a file() command API for SHA1, SHA224, SHA256, SHA384, and SHA512.
* Add file(MD5) command to compute cryptographic hashBrad King2011-11-161-0/+36
| | | | | Provide a CMake-language binding to the md5sum function previously available only by "cmake -E md5sum".
* Make file(DOWNLOAD) fail on http errorBrad King2011-09-191-0/+4
| | | | | | | | If a http server responds with a result code greater than 400 then the data returned from the download probably do not match that expected. Teach file(DOWNLOAD) to fail with an error in this case instead of silently pretending that the download worked. The file(UPLOAD) command already does this.
* Fix file() command descriptor leak on errorThomas Jarosch2011-09-021-0/+2
| | | | | | Credit goes to "cppcheck". Signed-off-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
* Teach file(DOWNLOAD|UPLOAD) to timeout after inactivityBrad King2011-06-011-0/+42
| | | | | Add option INACTIVITY_TIMEOUT to terminate the operation if there is no progress for more than a given amount of time.
* Merge topic 'require-cmake-2.6.3'Brad King2011-03-011-23/+7
|\ | | | | | | | | c3e452e Require at least CMake 2.6.3 to build current CMake
| * Require at least CMake 2.6.3 to build current CMakeBrad King2011-02-241-23/+7
| | | | | | | | Remove some cruft left for supporting builds with CMake 2.4.
* | Fix KWStyle line too long error (#11286)David Cole2011-02-281-1/+2
| |
* | Implement file(UPLOAD (#11286)David Cole2011-02-251-111/+306
|/ | | | Including documentation and testing, of course.
* Make file DOWNLOAD less noisy (#11761)David Cole2011-02-091-4/+0
| | | | Thanks to Daniel Nelson for the suggestion and the patch.
* file(DOWNLOAD): Fix error message formattingBrad King2010-09-101-58/+45
| | | | | | | The SetError method automatically starts the message with the name of the command. Fix up calls to it so that we get "file DOWNLOAD..." instead of "file FILE(DOWNLOAD ...". Also reduce length of long lines containing these calls.
* Merge branch 'file-write-umask'Brad King2010-06-071-12/+9
|\
| * Borland: No S_IWGRP is availableBrad King2010-06-031-0/+2
| |
| * Really trust umask in file(WRITE) command (#10789, #10126)Brad King2010-06-031-12/+7
| | | | | | | | | | | | | | | | | | Commit 8d0161c8 (Trust umask for file permissions, 2010-01-12) taught these commands to set permissions to 0666 explicitly. The intention was to let the open() call inside ofstream handle permsisions so that umask would be honored. Now we set permissions only when we need to preserve those on an existing file. New files will be created with umask-based permissions.
* | Merge branch 'improve-file-download'Brad King2010-06-071-16/+243
|\ \ | | | | | | | | | | | | Conflicts: Modules/ExternalProject.cmake
| * | Allow redirects: set CURLOPT_FOLLOWLOCATION to 1David Cole2010-06-041-0/+9
| | | | | | | | | | | | | | | | | | Enable file(DOWNLOAD ...) to follow redirects. Thanks to Michael Wild for requesting the addition and providing the majority of the patch.
| * | Fix unused variable warning in new code.David Cole2010-05-271-0/+3
| | |
| * | Improve FILE(DOWNLOAD) and ExternalProject.David Cole2010-05-271-16/+231
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve FILE(DOWNLOAD ...): - Add percent complete progress output to the FILE DOWNLOAD command. This progress output is off by default to preserve existing behavior. To turn it on, pass SHOW_PROGRESS as an argument. - Add EXPECTED_MD5 argument. Verify that the downloaded file has the expected md5 sum after download is complete. - Add documentation for SHOW_PROGRESS and EXPECTED_MD5. When the destination file exists already and has the expected md5 sum, then do not bother re-downloading the file. ("Short circuit" return.) Also, add a test that checks for the status output indicating that the short circuit behavior is actually occurring. Use a binary file for the test so that the md5 sum is guaranteed to be the same on all platforms regardless of "shifting text file line ending" issues. Improve ExternalProject: - Add argument URL_MD5. - Add verify step that compares md5 sum of .tar.gz file before extracting it. - Add md5 check to download step, too, to prevent unnecessary downloads. - Emit a warning message when a file is not verified. Indicate that the file may be corrupt or that no checksum was specified.
* | Avoid use of CopyAFile "copyPermissions" parameterBrad King2010-06-031-1/+8
|/ | | | | | | Commit 0fafdb7e (Do not copy permissions of files when making the copy in an install rule, 2008-12-18) added special behavior to KWSys file copy methods for this special case. Use a local solution to avoid use of the special behavior so it can be removed later.
* Handle non-ASCII terminators in file(STRINGS)Brad King2010-04-141-31/+19
| | | | | | | | | | Commit "Support more special characters in file(STRINGS)" (2009-10-06) attempted to support parsing strings from binaries produced by the Portland Group Fortran compiler. The compiler seems to put an extra byte just at the end of its string literals. Previously we dealt with this by explicitly enumerating bytes known to occur, but it seems that many such possibilities exist. Now we support extraction of strings that end in any non-ASCII character.
* Trust umask for file permissionsBrad King2010-01-121-4/+2
| | | | | Open output files with mode 0666 so that permissions are not more strict than umask permits. See issue #10126.
* Fix up download a bit, better error checking and uses of long not double for ↵Bill Hoffman2009-10-221-37/+124
| | | | timeout as curl needs, bug# 9748
* Fixed issues with message text in FILE command error situations. Added many ↵David Cole2009-10-171-8/+9
| | | | new test cases to increase the coverage of the FILE command even further.