On 05/14/2011 05:22 PM, Florin Ciubotaru wrote:
Hi,
This issue was addressed previously in http://jira.xwiki.org/jira/browse/XWIKI-6591, but other long path files have been committed since then. I feel the problem is going to continue to appear without a convention for directory naming.
Paths are very redundant in some cases, leading to long strings which are not compatible with windows systems.
This was originally voted in http://markmail.org/thread/xgb3lltoqfwphukk but the vote wasn't very strong. 3 voters actually preferred shorter names. I agree that this is a problem, and I'd propose to revert the original vote and use shorter names, without any redundancy. So:
Eg: - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-client/src/main/java/org/xwiki/extension/repository/xwiki/internal/
should become: xwiki-platform/core/extension/repositories/xwiki/client/src/main/java/org/xwiki/extension/repository/xwiki/internal/
- xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-server-api/src/main/java/org/xwiki/extension/repository/xwiki/internal/ - xwiki-platform-core/xwiki-platform-classloader/xwiki-platform-classloader-protocols/xwiki-platform-classloader-protocol-attachmentjar/src/test/java/org/xwiki/classloader/internal/protocol/attachmentjar/
We should probably have a convention limiting the git path(relative directory + filename) to a max of ~230 chars, leaving the developer with about 20 chars for his local root folder(which is still pretty low).
Current list of files breaking "git clone":
- xwiki-platform-core/xwiki-platform-classloader/xwiki-platform-classloader-protocols/xwiki-platform-classloader-protocol-attachmentjar/src/test/java/org/xwiki/classloader/internal/protocol/attachmentjar/AttachmentURLStreamHandlerTest.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-client/src/main/java/org/xwiki/extension/repository/xwiki/internal/XWikiExtension.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-client/src/main/java/org/xwiki/extension/repository/xwiki/internal/XWikiExtensionDependency.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-client/src/main/java/org/xwiki/extension/repository/xwiki/internal/XWikiExtensionRepository.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-client/src/main/java/org/xwiki/extension/repository/xwiki/internal/XWikiExtensionRepositoryFactory.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-server-api/src/main/java/org/xwiki/extension/repository/xwiki/internal/AbstractExtensionRESTResource.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-server-api/src/main/java/org/xwiki/extension/repository/xwiki/internal/ExtensionFileRESTResource.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-server-api/src/main/java/org/xwiki/extension/repository/xwiki/internal/ExtensionRESTResource.java - xwiki-platform-core/xwiki-platform-extension/xwiki-platform-extension-repositories/xwiki-platform-extension-repository-xwiki/xwiki-platform-extension-repository-xwiki-server-api/src/main/java/org/xwiki/extension/repository/xwiki/internal/ExtensionSimpleSearchRESTResource.java
-- Sergiu Dumitriu http://purl.org/net/sergiu/