[jira] [Commented] (JCRVLT-1) FileVault: Remove CQ/CRX/day/adobe references

["Ate Douma (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/JCRVLT-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14194820#comment-14194820 ] 

Ate Douma commented on JCRVLT-1:
--------------------------------

[~tripod] Tobias, are you sure? I only see a few test classes been updated in r1635841, but none of the classes referenced above.

> FileVault: Remove CQ/CRX/day/adobe references
> ---------------------------------------------
>
>                 Key: JCRVLT-1
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-1
>             Project: Jackrabbit FileVault
>          Issue Type: Bug
>            Reporter: Tobias Bocanegra
>            Assignee: Tobias Bocanegra
>            Priority: Minor
>             Fix For: 3.1.10
>
>
> as stated correctly by ate, we need to remove references to day/adobe and cq/crx defaults:
> > Most of it doesn't really do any 'harm' in a non-CQ5/CRX context, Like for
> > example the CRX specific Node types in (vault-core) DefaultNodeTypes.java, or
> > the default configurations in the (vault-core) defaultConfig-1.[0|1].xml files.
> > But in some other areas I think they might be(come) more than a nuisance, like
> > for example:
> > - default/fallback "crx.default" workspace name used in (vault-core)
> > AggregatManagerImpl.java
> > - default/fallback "/crx/server" prefix used in (vault-core) RepositoryAddress.java
> > - "/var/crxpatches" patchParentPath in (vault-core) ImportOptions.java
> > - CRX specific default URI/WPS constants in (vault-cli) VaultFsApp.java
> > - CRX specific constants in (vault-vlt) Sync.java
> > - and a few more (just search for CQ or CRX, case-insensitive)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)