[Insight-developers] Fwd: Change in ITK[master]: COMP: root of source and binary dir of ITK should not be too...

[Bradley Lowekamp]

On locked down PCs ( corporate or government ) some people can't change files out side of their home directory.

As far as I know windows also requires you to be an administer to be able to run/compile with visual studio. But perhaps this will change in the future?

I thought this was a bug in VS2010 and cmake 2.8.4 that was patched. That is it should be using shorter relative paths for some of these things.


Brad


On Jun 23, 2011, at 2:13 PM, David Cole wrote:

> This gerrit change proposes to limit the path length of both the source and binary trees on Windows builds of ITK. Before we accept this change, is there anybody who has a reason why this would be a problem for them?
> 
> I think it's a reasonable compromise given the length of the depth of the tree, and the problems that ensue. (When using certain versions of CMake and certain versions of Visual Studio, relative path names combined with the absolute path of the build tree, combine to exceed the 250-something character file name limit in some of the underlying Windows tools used...)
> 
> Raise objections now, if any, or forever hold your peace. :-)
> 
> 
> Thanks,
> David C.
> 
> 
> ---------- Forwarded message ----------
> From: Code Review <review at kitware.com>
> Date: Thu, Jun 23, 2011 at 12:07 PM
> Subject: Change in ITK[master]: COMP: root of source and binary dir of ITK should not be too...
> To: David Cole <david.cole at kitware.com>
> 
> 
> From Alex. Gouaillard <agouaillard at gmail.com>:
> 
> Hello David Cole,
> 
> I'd like you to do a code review.  Please visit
> 
>    http://review.source.kitware.com/1967
> 
> to review the following change.
> 
> Change subject: COMP: root of source and binary dir of ITK should not be too deep.
> ......................................................................
> 
> COMP: root of source and binary dir of ITK should not be too deep.
> 
> Change-Id: I79013671626f70b869e5d4e11a7e1d8987dad2b9
> ---
> M CMakeLists.txt
> 1 file changed, 23 insertions(+), 0 deletions(-)
> 
> 
>  git pull ssh://review.source.kitware.com/ITK refs/changes/67/1967/1
> --
> To view, visit http://review.source.kitware.com/1967
> To unsubscribe, visit http://review.source.kitware.com/settings
> 
> Gerrit-MessageType: newchange
> Gerrit-Change-Id: I79013671626f70b869e5d4e11a7e1d8987dad2b9
> Gerrit-PatchSet: 1
> Gerrit-Project: ITK
> Gerrit-Branch: master
> Gerrit-Owner: Alex. Gouaillard <agouaillard at gmail.com>
> Gerrit-Reviewer: David Cole <david.cole at kitware.com>
> 
> _______________________________________________
> Powered by www.kitware.com
> 
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
> 
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.html
> 
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
> 
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers

========================================================
Bradley Lowekamp  
Lockheed Martin Contractor for
Office of High Performance Computing and Communications
National Library of Medicine 
blowekamp at mail.nih.gov


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110623/495a1a18/attachment.htm>