239 Commits (2cecd73aad272e1ecd5455614db765e699bd33da)

Author	SHA1	Message	Date
Josh Soref	5178dd36e8	spelling: regular	8 years ago
Anthony Allen	f3c361de6b	Fully qualify hyper-v powershell commands	8 years ago
SwampDragons	7d5d62d748	Merge pull request #2906 from arizvisa/GH-2377 ... Improved support for downloading and validating a uri containing a Windows UNC path or a relative file:// scheme	8 years ago
Ali Rizvi-Santiago	9eb2f37429	Ack! Forgot to include the test-fixtures/SomeDir/myfile.txt file...	8 years ago
Ali Rizvi-Santiago	d4b00b722a	Removed an extra '/' from the TestFileExistsLocally test in common/config_test.go	8 years ago
Ali Rizvi-Santiago	efc97dbda2	Fixed TestFileExistsLocally tests in common/config_test.go so that they're actually being run. Added a non-existent-protocol:// test.	8 years ago
Ali Rizvi-Santiago	c366a1e160	Inverted the logic of FileExistsLocally as suggested by @SwampDragons as remote URLs are assumed to exist locally.	8 years ago
Ali Rizvi-Santiago	c98a074f0d	Renamed common/config.go's SupportedURL to SupportedProtocol as suggested by @SwampDragons.	8 years ago
Ben Phegan	3c88e787df	Merge branch 'master' into hyperv_mac_address	8 years ago
Matthew Hooker	8cd403425e	test fixes WIP	8 years ago
Matthew Hooker	5d48d658b4	Wire context through misc steps ... Some steps actually need to pass the context around, so let's create a ctx variable and pass it.	8 years ago
Matthew Hooker	7a189a83a1	fix imports ... `find . -type f -name '*.go' -not -path "./vendor/*" -exec goimports -w {} \;`	8 years ago
Matthew Hooker	a831d522be	change run signatures ... Run now takes a context as well as a statebag. We'll assign the context to the blank identifier to prevent namespace collisions. We'll let the step authors opt-in to using the context. `find . -iname "step_*.go" -exec gsed -i'' 's/func \(.*\)Run(/func \1Run(_ context.Context, /' {} \;`	8 years ago
Matthew Hooker	366dc3da0a	move multistep imports to helper. ... gomvpkg -from "github.com/mitchellh/multistep" -to "github.com/hashicorp/packer/helper/multistep"	8 years ago
Matthew Hooker	ebe995c0ff	run goimports	8 years ago
Ali Rizvi-Santiago	0e0b467da7	Forgot to check some errors during the adding of files to the floppy disk. This gives users some better information in case packer is unable to add a file...like if there's not enough disk space available.	8 years ago
Ali Rizvi-Santiago	97fc9c02a5	Grr...missed the case that actually mattered on linux.	8 years ago
Ali Rizvi-Santiago	f9572cb244	Fixed a bug on linux related to forgetting to check the platform for the forward-slash prefix.	8 years ago
Ali Rizvi-Santiago	41f4dc3f3d	umm...gofmt -w on common/config{,_test}.go from linux instead of windows(?)	8 years ago
Ali Rizvi-Santiago	7cd5d576d9	Updated common/config.go's FileExistsLocally implementation to use the LocalDownloader interface for determining the real file path.	8 years ago
Ali Rizvi-Santiago	8a102a42a0	gofmt -w on common/config{,_test}.go	8 years ago
Ali Rizvi-Santiago	15079a99dc	Fixed common/config_test.go tests for DownloadableURL to avoid writing to disk on the windows platform. Also added tests for relative paths/uris. ... common/config_test.go: Replaced instances of os.Mkdir and os.Create with tests that use the existing "common/test-fixtures" mechanism. Removed the runtime.GOOS test for the "FileExistsLocally" test, as the functionality should work regardless of the platform. Added some more comprehensive tests for the relative uri/pathing. Replaced the Windows Object Manager name test as the Object Manager's naming scheme is different from a UNC path. Modified the FilePaths tests to support the policy of windows absolute paths being prefixed with the `/` introduced with PR #5761.	8 years ago
Ali Rizvi-Santiago	95f60f6153	Modified common/config.go to accommodate some of the new DownloadableURL policies made by the PR #5761 merge. ... common/config.go: Added the ability for DownloadableURL to promote UNC paths to the SMB uri. Modified DownloadableURL to include the "./" prefix when a relative path is passed to it. Fix-up the DownloadableURL argument if on windows and incorrectly prefixed with "/".	8 years ago
Ali Rizvi-Santiago	50e9cd2ca7	Initial fixes of common/config.go after rebase before refactoring of test-cases so that they don't require root to run.	8 years ago
Ali Rizvi-Santiago	c17f827e1d	Split up DownloadableURL() into it's individual components: SupportedURL(), DownloadableURL(), and ValidatedURL(). Updated all instances of DownloadableURL() to point to ValidatedURL(). Reverted the tests that are based on un-supported protocols.	8 years ago
Ali Rizvi-Santiago	3cf448f6ec	Reverted previously removed additions of tests that check for ftp:// or nonexistent-protocol:// using DownloadableURL. DownloadableURL's responsibility is not to have inherent knowledge of protocols that are available, but to format an invalid url/path to a valid url/path.	8 years ago
Ali Rizvi-Santiago	4a1fb0d262	Grrr...gofmt -w common/*.go	8 years ago
Ali Rizvi-Santiago	5d97b105a8	Removed implementation of the ftp protocol and the usage of cheggaaa's progress-bar as suggested by @SwampDragons. Replaced some of the old smoke-tests that were based on the ftp-protocol non-existing with a "non-existent://" protocol that's guaranteed to not exist.	8 years ago
Ali Rizvi-Santiago	5a3e98b529	Updated the testcases in common/download_test.go to pass a non-nil progress-bar due to the removal of a pointer type in commit ed2e341b7d7f49a063dd5018701b4ae548b8ec14 from yesterday.	8 years ago
Ali Rizvi-Santiago	ab4490b967	Consolidated progress bar's appearance into the GetDefaultProgressBar() function. Updated dependency for cheggaaa's progress-bar from the gopkg.in location to the better maintained one on github.com.	8 years ago
Ali Rizvi-Santiago	8c6efe336c	Added second argument for custom-formatted progress-bar to NewDownloadClient in common/download_test.go. This second parameter was added as a result of commit f0bd9018f3e318caafb1fe7d46e04c470e07c092 which lets you customize the progress-bar format.	8 years ago
Ali Rizvi-Santiago	69e5eec1ce	Consolidated the progress-bar's format into common/step_download.go. Removed DownloadClient's PercentProgress callback since cheggaaa's progress-bar already does that.	8 years ago
Ali Rizvi-Santiago	d85883582f	Changed a critical error to a non-critical one when dealing with the strange .CopyFile flag in common/download.go.	8 years ago
Ali Rizvi-Santiago	c978e27f0f	grr. removed an assignment that was dead in common/download.go.	8 years ago
Ali Rizvi-Santiago	5a4ce2165c	Modified common/download_test.go to not test the smb:// uri on platforms other than windows. Added an immediate platform error to SMBDownloader.Download as opposed to letting .toPath return it (which would have left the structure partially initialized).	8 years ago
Ali Rizvi-Santiago	4783b6508e	Fix common/download_test.go to avoid formatting the volume name to a hidden windows share when not on windows.	8 years ago
Ali Rizvi-Santiago	b1ff14714b	go fmt	8 years ago
Ali Rizvi-Santiago	11ff4439a6	Moved the setting of HTTPDownloader's current progress to after the object actually gets instantiated. ;)	8 years ago
Ali Rizvi-Santiago	0fa6c3782e	Added a progressbar using gopkg.in/cheggaaa/pb.v1 as per #3578 for all the DownloadClients in common/download.go.	8 years ago
Ali Rizvi-Santiago	2f1104625d	Fixed some of the unit-tests in common/ due to the changes made in {config,download}.go ... config.go: Fixed some issues related to the url scheme not being lowercased which broke some of the tests. config_test.go: Removed the UNC share test for \\host\share\file since SMB support has been moved to a different uri scheme. download_test.go: Explicitly set the CopyFile configuration option for all the unit-tests that test file copying capability. Removed the UNC share testcase since it's under a different uri scheme now. Modified the file:// UNC share testcase to explicitly test the smb:// uri. Changed the incorrect t.Errorf calls to t.Logf so that the tests can pass.	8 years ago
Ali Rizvi-Santiago	6170e24ecb	Refactored the code a bit to move the CopyFile hack out of DownloadClient and instead into each protocol. ... config.go: Removed all of the windows-specific net/url hackery since it's now handled mostly by download.go Removed the replacement of '\' with '/' since url.Parse does it now. Added knowledge of the other protocols implemented in download.go (ftp, smb) Removed some modules that were unused in this commit. download.go: Moved the file-path conversions for the different protocols into their own internally callable functions. Shuffled some of the functions around in case someone wants to implement the ability to resume. Modified DownloadClient.Get to remove the CopyFile special case and trust the protocol implementations if a user doesn't want to copy the file. Since all the protocols except for HTTPDownloader implement Cancel, added a Resume method as a placeholder for another developer to implement. Added a few missing names from their function definitions. Fixed the syntax in a few lines due to my suckage at go. Adjusted the types for progress and total so that they support 64-bit sizes. Removed the usage of the bufio library since it wasn't really being used.	8 years ago
Ali Rizvi-Santiago	60831801a7	Added the file, ftp, and smb downloaders to common/download.go	8 years ago
Ali Rizvi-Santiago	da9c94b345	Added some testcases for the various file uri transforms to download_test.go ... Moved some of the code for normalizing a Windows file uri to a regular path into it's own function NormalizeWindowsURL	8 years ago
Ali Rizvi-Santiago	281dd1258a	Added proper support for downloading via a Windows UNC path or a relative uri. ... Added proper support for validating a downloadableURL containing a UNC or relative uri. Removed the workaround for an earlier Go issue that had remained dormant in common/download.go (issue #5927). When building a .vmx file via the vmware-iso builder, transform the path to the correct os-formatted one (using filepath.FromSlash).	8 years ago
SwampDragons	fb730cf521	Merge pull request #5761 from hashicorp/fix_5713 ... fix nasty edge case where we can't find guest additions on windows if they are on a different drive	8 years ago
Megan Marsh	bdd186fa2b	add tests for fileexistslocally helper function	8 years ago
Megan Marsh	3ace5bb91b	simplify FileExistsLocally	8 years ago
Megan Marsh	55ddbf4765	sloppy copypasta	8 years ago
Megan Marsh	898dadd53c	re-add this block. I still don't think we need it but I don't want to risk breaking things with this bugfix.	8 years ago
Megan Marsh	a04a921c2d	add UNC path to test cases, so I can try to enable it in future	8 years ago