You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Wayne Fay (JIRA)" <ji...@codehaus.org> on 2008/03/17 16:41:48 UTC

[jira] Created: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

FileSet needs a toString() method to properly print in debug mode
-----------------------------------------------------------------

                 Key: MNG-3468
                 URL: http://jira.codehaus.org/browse/MNG-3468
             Project: Maven 2
          Issue Type: Improvement
    Affects Versions: 2.0.8
            Reporter: Wayne Fay
         Attachments: fileset.patch

It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 

Before this change, it prints like: 
FileSet@129f3b5

With this patch, it prints like (assuming you accept my PatternSet patch):
FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Re: [jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by Vincent Siveton <vi...@gmail.com>.
Agree with you, Paul.

Vincent

2008/6/29, Paul Benedict <pb...@apache.org>:
> I think it is very smart to always mark issues that are both 2.0.x and 2.1
>  (HEAD). While it my be "common knowledge" among the depths Maven developers
>  that issues being applied to 2.0.x are usually applied to trunk as well, it
>  shouldn't be assumed when managing issues. It's possible for issues to be
>  2.0.x only, 2.1 only, and where they are back/forward-ported to both. I
>  think it's good housekeeping to always say both to keep track of the changes
>  in both branches.
>
>
>  Paul
>
>
>  On Sun, Jun 29, 2008 at 4:57 PM, Brett Porter <br...@apache.org> wrote:
>
>  > Vincent,
>  >
>  > I'm a bit confused about these changes. It should be obvious that if it
>  > were fixed in 2.0.9 that it is also present (if relevant) in 2.1... won't it
>  > be confusing to see the 2.1 release notes and see a whole lot of things that
>  > are essentially not different from the last 2.0.x releases?
>  >
>  > Thanks,
>  > Brett
>  >
>  > On 28/06/2008, at 10:46 PM, Vincent Siveton (JIRA) wrote:
>  >
>  >
>  >>    [
>  >> http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
>  >>  ]
>  >>
>  >> Vincent Siveton updated MNG-3468:
>  >> ---------------------------------
>  >>
>  >>   Fix Version/s: 2.1-alpha-1
>  >>
>  >>  FileSet needs a toString() method to properly print in debug mode
>  >>> -----------------------------------------------------------------
>  >>>
>  >>>               Key: MNG-3468
>  >>>               URL: http://jira.codehaus.org/browse/MNG-3468
>  >>>           Project: Maven 2
>  >>>        Issue Type: Improvement
>  >>>  Affects Versions: 2.0.8
>  >>>          Reporter: Wayne Fay
>  >>>          Assignee: Brian Fox
>  >>>           Fix For: 2.0.9, 2.1-alpha-1
>  >>>
>  >>>       Attachments: fileset.patch
>  >>>
>  >>>
>  >>> It would be nice if FileSet had a toString() method so it would print
>  >>> something intelligent rather than simply a memory location in debug mode.
>  >>> Before this change, it prints like:
>  >>> FileSet@129f3b5
>  >>> With this patch, it prints like (assuming you accept my PatternSet
>  >>> patch):
>  >>> FileSet {directory: src/main/resources, PatternSet [includes: {},
>  >>> excludes: {}]}
>  >>>
>  >>
>  >> --
>  >> This message is automatically generated by JIRA.
>  >> -
>  >> If you think it was sent incorrectly contact one of the administrators:
>  >> http://jira.codehaus.org/secure/Administrators.jspa
>  >> -
>  >> For more information on JIRA, see: http://www.atlassian.com/software/jira
>  >>
>  >>
>  >>
>  > --
>  > Brett Porter
>  > brett@apache.org
>  > http://blogs.exist.com/bporter/
>  >
>  >
>  > ---------------------------------------------------------------------
>  > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>  > For additional commands, e-mail: dev-help@maven.apache.org
>  >
>  >
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: [jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by Paul Benedict <pb...@apache.org>.
I think it is very smart to always mark issues that are both 2.0.x and 2.1
(HEAD). While it my be "common knowledge" among the depths Maven developers
that issues being applied to 2.0.x are usually applied to trunk as well, it
shouldn't be assumed when managing issues. It's possible for issues to be
2.0.x only, 2.1 only, and where they are back/forward-ported to both. I
think it's good housekeeping to always say both to keep track of the changes
in both branches.

Paul

On Sun, Jun 29, 2008 at 4:57 PM, Brett Porter <br...@apache.org> wrote:

> Vincent,
>
> I'm a bit confused about these changes. It should be obvious that if it
> were fixed in 2.0.9 that it is also present (if relevant) in 2.1... won't it
> be confusing to see the 2.1 release notes and see a whole lot of things that
> are essentially not different from the last 2.0.x releases?
>
> Thanks,
> Brett
>
> On 28/06/2008, at 10:46 PM, Vincent Siveton (JIRA) wrote:
>
>
>>    [
>> http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
>>  ]
>>
>> Vincent Siveton updated MNG-3468:
>> ---------------------------------
>>
>>   Fix Version/s: 2.1-alpha-1
>>
>>  FileSet needs a toString() method to properly print in debug mode
>>> -----------------------------------------------------------------
>>>
>>>               Key: MNG-3468
>>>               URL: http://jira.codehaus.org/browse/MNG-3468
>>>           Project: Maven 2
>>>        Issue Type: Improvement
>>>  Affects Versions: 2.0.8
>>>          Reporter: Wayne Fay
>>>          Assignee: Brian Fox
>>>           Fix For: 2.0.9, 2.1-alpha-1
>>>
>>>       Attachments: fileset.patch
>>>
>>>
>>> It would be nice if FileSet had a toString() method so it would print
>>> something intelligent rather than simply a memory location in debug mode.
>>> Before this change, it prints like:
>>> FileSet@129f3b5
>>> With this patch, it prints like (assuming you accept my PatternSet
>>> patch):
>>> FileSet {directory: src/main/resources, PatternSet [includes: {},
>>> excludes: {}]}
>>>
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> If you think it was sent incorrectly contact one of the administrators:
>> http://jira.codehaus.org/secure/Administrators.jspa
>> -
>> For more information on JIRA, see: http://www.atlassian.com/software/jira
>>
>>
>>
> --
> Brett Porter
> brett@apache.org
> http://blogs.exist.com/bporter/
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>

Re: [jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by Vincent Siveton <vi...@gmail.com>.
Hi Brett,

2008/6/29, Brett Porter <br...@apache.org>:
> Vincent,
>
>  I'm a bit confused about these changes. It should be obvious that if it
> were fixed in 2.0.9 that it is also present (if relevant) in 2.1... won't it

Since the changes have not been applied initially, it is not so obvious ;)

> be confusing to see the 2.1 release notes and see a whole lot of things that
> are essentially not different from the last 2.0.x releases?

IMHO I think it is important to specify both since we have a branch.
Some changes could be 2.0.x specific only.

I didn't remember a thread about some jira rules so maybe we need to
clarify them. Thanks to point that!

Cheers,

Vincent

>  Thanks,
>  Brett
>
>
>  On 28/06/2008, at 10:46 PM, Vincent Siveton (JIRA) wrote:
>
>
> >
> >    [
> http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
> ]
> >
> > Vincent Siveton updated MNG-3468:
> > ---------------------------------
> >
> >   Fix Version/s: 2.1-alpha-1
> >
> >
> > > FileSet needs a toString() method to properly print in debug mode
> > >
> -----------------------------------------------------------------
> > >
> > >               Key: MNG-3468
> > >               URL:
> http://jira.codehaus.org/browse/MNG-3468
> > >           Project: Maven 2
> > >        Issue Type: Improvement
> > >  Affects Versions: 2.0.8
> > >          Reporter: Wayne Fay
> > >          Assignee: Brian Fox
> > >           Fix For: 2.0.9, 2.1-alpha-1
> > >
> > >       Attachments: fileset.patch
> > >
> > >
> > > It would be nice if FileSet had a toString() method so it would print
> something intelligent rather than simply a memory location in debug mode.
> > > Before this change, it prints like:
> > > FileSet@129f3b5
> > > With this patch, it prints like (assuming you accept my PatternSet
> patch):
> > > FileSet {directory: src/main/resources, PatternSet [includes: {},
> excludes: {}]}
> > >
> >
> > --
> > This message is automatically generated by JIRA.
> > -
> > If you think it was sent incorrectly contact one of the administrators:
> http://jira.codehaus.org/secure/Administrators.jspa
> > -
> > For more information on JIRA, see:
> http://www.atlassian.com/software/jira
> >
> >
> >
>
>  --
>  Brett Porter
>  brett@apache.org
>  http://blogs.exist.com/bporter/
>
>
> ---------------------------------------------------------------------
>  To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>  For additional commands, e-mail: dev-help@maven.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: [jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by Brett Porter <br...@apache.org>.
Vincent,

I'm a bit confused about these changes. It should be obvious that if  
it were fixed in 2.0.9 that it is also present (if relevant) in 2.1...  
won't it be confusing to see the 2.1 release notes and see a whole lot  
of things that are essentially not different from the last 2.0.x  
releases?

Thanks,
Brett

On 28/06/2008, at 10:46 PM, Vincent Siveton (JIRA) wrote:

>
>     [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel 
>  ]
>
> Vincent Siveton updated MNG-3468:
> ---------------------------------
>
>    Fix Version/s: 2.1-alpha-1
>
>> FileSet needs a toString() method to properly print in debug mode
>> -----------------------------------------------------------------
>>
>>                Key: MNG-3468
>>                URL: http://jira.codehaus.org/browse/MNG-3468
>>            Project: Maven 2
>>         Issue Type: Improvement
>>   Affects Versions: 2.0.8
>>           Reporter: Wayne Fay
>>           Assignee: Brian Fox
>>            Fix For: 2.0.9, 2.1-alpha-1
>>
>>        Attachments: fileset.patch
>>
>>
>> It would be nice if FileSet had a toString() method so it would  
>> print something intelligent rather than simply a memory location in  
>> debug mode.
>> Before this change, it prints like:
>> FileSet@129f3b5
>> With this patch, it prints like (assuming you accept my PatternSet  
>> patch):
>> FileSet {directory: src/main/resources, PatternSet [includes: {},  
>> excludes: {}]}
>
> -- 
> This message is automatically generated by JIRA.
> -
> If you think it was sent incorrectly contact one of the  
> administrators: http://jira.codehaus.org/secure/Administrators.jspa
> -
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
>

--
Brett Porter
brett@apache.org
http://blogs.exist.com/bporter/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by "Vincent Siveton (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vincent Siveton updated MNG-3468:
---------------------------------

    Fix Version/s: 2.1-alpha-1

> FileSet needs a toString() method to properly print in debug mode
> -----------------------------------------------------------------
>
>                 Key: MNG-3468
>                 URL: http://jira.codehaus.org/browse/MNG-3468
>             Project: Maven 2
>          Issue Type: Improvement
>    Affects Versions: 2.0.8
>            Reporter: Wayne Fay
>            Assignee: Brian Fox
>             Fix For: 2.0.9, 2.1-alpha-1
>
>         Attachments: fileset.patch
>
>
> It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 
> Before this change, it prints like: 
> FileSet@129f3b5
> With this patch, it prints like (assuming you accept my PatternSet patch):
> FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Closed: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by "Brian Fox (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Fox closed MNG-3468.
--------------------------

         Assignee: Brian Fox
       Resolution: Fixed
    Fix Version/s: 2.0.10

> FileSet needs a toString() method to properly print in debug mode
> -----------------------------------------------------------------
>
>                 Key: MNG-3468
>                 URL: http://jira.codehaus.org/browse/MNG-3468
>             Project: Maven 2
>          Issue Type: Improvement
>    Affects Versions: 2.0.8
>            Reporter: Wayne Fay
>            Assignee: Brian Fox
>             Fix For: 2.0.10
>
>         Attachments: fileset.patch
>
>
> It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 
> Before this change, it prints like: 
> FileSet@129f3b5
> With this patch, it prints like (assuming you accept my PatternSet patch):
> FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by "Wayne Fay (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_127861 ] 

Wayne Fay commented on MNG-3468:
--------------------------------

I spoke with Brian about these, and the patches will go into 2.0.10 unless 2.0.9 is re-spun for some reason.

So, unless the fix version indicated in this jira is wrong, I think this will have to wait for 2.0.10's release notes.

> FileSet needs a toString() method to properly print in debug mode
> -----------------------------------------------------------------
>
>                 Key: MNG-3468
>                 URL: http://jira.codehaus.org/browse/MNG-3468
>             Project: Maven 2
>          Issue Type: Improvement
>    Affects Versions: 2.0.8
>            Reporter: Wayne Fay
>            Assignee: Brian Fox
>             Fix For: 2.0.10
>
>         Attachments: fileset.patch
>
>
> It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 
> Before this change, it prints like: 
> FileSet@129f3b5
> With this patch, it prints like (assuming you accept my PatternSet patch):
> FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by "Brian Fox (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Fox updated MNG-3468:
---------------------------

    Fix Version/s:     (was: 2.0.10)
                   2.0.9

> FileSet needs a toString() method to properly print in debug mode
> -----------------------------------------------------------------
>
>                 Key: MNG-3468
>                 URL: http://jira.codehaus.org/browse/MNG-3468
>             Project: Maven 2
>          Issue Type: Improvement
>    Affects Versions: 2.0.8
>            Reporter: Wayne Fay
>            Assignee: Brian Fox
>             Fix For: 2.0.9
>
>         Attachments: fileset.patch
>
>
> It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 
> Before this change, it prints like: 
> FileSet@129f3b5
> With this patch, it prints like (assuming you accept my PatternSet patch):
> FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MNG-3468) FileSet needs a toString() method to properly print in debug mode

Posted by "Paul Benedict (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MNG-3468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_127851 ] 

Paul Benedict commented on MNG-3468:
------------------------------------

Please update these 3 related tickets to 2.0.9 so they get on the release notes.

> FileSet needs a toString() method to properly print in debug mode
> -----------------------------------------------------------------
>
>                 Key: MNG-3468
>                 URL: http://jira.codehaus.org/browse/MNG-3468
>             Project: Maven 2
>          Issue Type: Improvement
>    Affects Versions: 2.0.8
>            Reporter: Wayne Fay
>            Assignee: Brian Fox
>             Fix For: 2.0.10
>
>         Attachments: fileset.patch
>
>
> It would be nice if FileSet had a toString() method so it would print something intelligent rather than simply a memory location in debug mode. 
> Before this change, it prints like: 
> FileSet@129f3b5
> With this patch, it prints like (assuming you accept my PatternSet patch):
> FileSet {directory: src/main/resources, PatternSet [includes: {}, excludes: {}]}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira