You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@forrest.apache.org by "Gavin (JIRA)" <ji...@apache.org> on 2007/07/20 14:34:06 UTC

[jira] Commented: (FOR-812) Remove dependency of projectInfo on skinconf.xml

    [ https://issues.apache.org/jira/browse/FOR-812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514189 ] 

Gavin commented on FOR-812:
---------------------------

Cyriaque, is your fix suitable to be added now ?

> Remove dependency of projectInfo on skinconf.xml
> ------------------------------------------------
>
>                 Key: FOR-812
>                 URL: https://issues.apache.org/jira/browse/FOR-812
>             Project: Forrest
>          Issue Type: Sub-task
>          Components: Plugin: input.projectInfo
>    Affects Versions: 0.8
>            Reporter: Ross Gardler
>            Priority: Blocker
>             Fix For: 0.9-dev
>
>
> The dispatcher will remove the file skinconf.xml. However, the projectInfo plugin uses it to get the URL for the issue tracker.
> We should chage the projectInfo plugin so that this value is provided by a plugin property.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Commented: (FOR-812) Remove dependency of projectInfo on skinconf.xml

Posted by Ross Gardler <rg...@apache.org>.
Cyriaque Dupoirieux wrote:
> le 20/07/2007 14:34 Gavin (JIRA) a écrit :
>>     [ https://issues.apache.org/jira/browse/FOR-812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514189 ] 
>>
>> Gavin commented on FOR-812:
>> ---------------------------
>>
>> Cyriaque, is your fix suitable to be added now ?
>>   
> Yes it should still work even if it is one year old :-) .
> Beware, we had a deep discution on this and I don't know what to think
> of it now :
> http://marc.theaimsgroup.com/?l=forrest-dev&w=2&r=1&q=b&s=430032)
>

That thread was about releasing a 0.8 plugin that depends on (at the 
time) alpha code in core (the forrest.properties.xml). This code can be 
considered beta now, it is in the 0.8 release, and some of the -dev 
plugins use it. Furthermore, it is scheduled to be included in 0.9.

In addition, as part of the 0.8 release cycle we packaged and released a 
0.8 version of the plugin without this commit. So the main reason for my 
-1 on this has been removed (see thread above for discussion).

I'm happy to withdraw my -1 on the understanding that the version number 
for that the minimum forrest version of the plugin becomes 0.9 as a 
result of this change.

Ross

Re: [jira] Commented: (FOR-812) Remove dependency of projectInfo on skinconf.xml

Posted by Cyriaque Dupoirieux <Cy...@pcotech.fr>.
le 20/07/2007 14:34 Gavin (JIRA) a écrit :
>     [ https://issues.apache.org/jira/browse/FOR-812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514189 ] 
>
> Gavin commented on FOR-812:
> ---------------------------
>
> Cyriaque, is your fix suitable to be added now ?
>   
Yes it should still work even if it is one year old :-) .
Beware, we had a deep discution on this and I don't know what to think
of it now :
http://marc.theaimsgroup.com/?l=forrest-dev&w=2&r=1&q=b&s=430032)

Salutations,
Cyriaque,

>   
>> Remove dependency of projectInfo on skinconf.xml
>> ------------------------------------------------
>>
>>                 Key: FOR-812
>>                 URL: https://issues.apache.org/jira/browse/FOR-812
>>             Project: Forrest
>>          Issue Type: Sub-task
>>          Components: Plugin: input.projectInfo
>>    Affects Versions: 0.8
>>            Reporter: Ross Gardler
>>            Priority: Blocker
>>             Fix For: 0.9-dev
>>
>>
>> The dispatcher will remove the file skinconf.xml. However, the projectInfo plugin uses it to get the URL for the issue tracker.
>> We should chage the projectInfo plugin so that this value is provided by a plugin property.
>>     
>
>