Bug 30482 - Plus "+" characters in paths are converted to spaces in the "C:/" build cache folder
Summary: Plus "+" characters in paths are converted to spaces in the "C:/" build cache...
Status: RESOLVED NOT_ON_ROADMAP
Alias: None
Product: Visual Studio Extensions
Classification: Xamarin
Component: iOS ()
Version: 3.11 (C5)
Hardware: PC Windows
: --- minor
Target Milestone: ---
Assignee: Bugzilla
URL:
Depends on:
Blocks:
 
Reported: 2015-05-26 19:31 UTC by Brendan Zagaeski (Xamarin Team, assistant)
Modified: 2015-10-19 11:06 UTC (History)
4 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:


Attachments
Test case (12.70 KB, application/zip)
2015-05-26 19:31 UTC, Brendan Zagaeski (Xamarin Team, assistant)
Details
Logs and verbose version info (39.34 KB, application/zip)
2015-05-26 19:32 UTC, Brendan Zagaeski (Xamarin Team, assistant)
Details


Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
RESOLVED NOT_ON_ROADMAP

Description Brendan Zagaeski (Xamarin Team, assistant) 2015-05-26 19:31:57 UTC
Created attachment 11346 [details]
Test case

Plus "+" characters in paths are converted to spaces in the "C:/" build cache folder


This does not appear to have any negative consequences on the build, but I'm also not sure of all the circumstances when the "C:/" build cache folder can possibly be used. If it is not _possible_ that this change from "+" characters to spaces could _ever_ cause a problem in the build, then I guess this bug is ignorable.




## Regression status: NOT a recent regression

### BAD
Windows:    XamarinVS    3.9.547.0  (20fd2f0)
Build host: Xamarin.iOS  8.10.0.267 (6481535)

### BAD
Windows:    XamarinVS    3.11.576.0 (b28d0a1)
Build host: Xamarin.iOS  8.10.1.59  (13d1cdc)




## Steps to reproduce

Build the attached test case in Visual Studio (tested in the "Debug|iPhone" configuration). (This is just an iOS app template project with the Json.NET NuGet pacakge added.)




## Results

The copied "C:\...\portable-net40+sl5+wp80+win8+wpa81\" directory replaces the "+" characters with spaces:

> $ find ~/Library/Caches/Xamarin/mtbs/builds/UnifiedSingleViewIphone1/18a2c8070c0e10416452a33a14d825e4/C\: -name "Newtonsoft.Json.dll"
> /Users/macuser/Library/Caches/Xamarin/mtbs/builds/UnifiedSingleViewIphone1/18a2c8070c0e10416452a33a14d825e4/C:/source/UnifiedSingleViewIphone1/packages/Newtonsoft.Json.6.0.8/lib/portable-net40 sl5 wp80 win8 wpa81/Newtonsoft.Json.dll




## Expected results

I would expect the copied "portable-net40+sl5+wp80+win8+wpa81" folder in the build cache to keep the "+" characters intact.




## Additional version information

Microsoft Visual Studio Professional 2013
Version 12.0.31101.00 Update 4
Microsoft .NET Framework
Version 4.6.00057


### OS X 10.10.3

Mono 4.0.1 ((detached/11b5830)
Xcode 6.3.1 (7703), Build 6D1002
Comment 1 Brendan Zagaeski (Xamarin Team, assistant) 2015-05-26 19:32:39 UTC
Created attachment 11347 [details]
Logs and verbose version info
Comment 2 Brendan Zagaeski (Xamarin Team, assistant) 2015-05-27 16:43:23 UTC
One little typo correction on the version information. When testing with XVS 3.9.547.0, I was of course using 8.9.1.3 on the build host (otherwise I would not have been able to pair).

### BAD
Windows:    XamarinVS    3.9.547.0  (20fd2f0)
Build host: Xamarin.iOS  8.9.1.3    (f7736a4)
Comment 3 Joaquin Jares 2015-10-19 11:06:38 UTC
That path has no consequence and the build works. Closing as not in roadmap. Side note: generally speaking, that folder shouldn't even exist in the mac. We do need some of those, but it's highly likely that we'll completely remove it soon.