Btw Imran, if you find such things then please file a Jira yourself so we'd 
also have tracked your highly appreciated help as a git pro 
:)

LieGrue,
strub

--- Mark Struberg <strub...@yahoo.de> schrieb am Mo, 13.4.2009:

> Von: Mark Struberg <strub...@yahoo.de>
> Betreff: Re: [PATCH] Git Change log takes current path into account
> An: scm-dev@maven.apache.org
> Datum: Montag, 13. April 2009, 11:22
> 
> Hi Imran!
> 
> Sorry that it took so long, but I was busy.
> 
> > I would be grateful if you would kindly state an
> example
> > where it
> > would not be sufficient with the change and was
> without
> > it.
> 
> Benjamin pointed me to a page [1] where all those pitfalls
> are listed.
> 
> So what I'd do is to add a
> -- ${basedir} 
> to the git-whatchanged command.
> 
> I created a Jira [2] and attached my changes as patch.
> 
> Please note that I've only tested this on Linux, so I'd be
> glad if someone can test this on Win also? - txs!
> 
> 
> LieGrue,
> strub
> 
> [1] 
> http://maven.apache.org/plugin-developers/common-bugs.html#Resolving_Relative_Paths
> [2] http://jira.codehaus.org/browse/SCM-458
> 
> --- Imran M Yousuf <imyou...@gmail.com>
> schrieb am Di, 7.4.2009:
> 
> > Von: Imran M Yousuf <imyou...@gmail.com>
> > Betreff: Re: [PATCH] Git Change log takes current path
> into account
> > An: "Mark Struberg" <strub...@yahoo.de>
> > CC: scm-dev@maven.apache.org
> > Datum: Dienstag, 7. April 2009, 10:32
> > On Tue, Apr 7, 2009 at 1:42 PM, Mark
> > Struberg <strub...@yahoo.de>
> > wrote:
> > >
> > > Hi Imran!
> > >
> > > What is the special usecase for this?
> > > Maybe multi-module builds where the current pom
> is
> > only 1 sub-module of the whole build?
> > >
> > 
> > It is not a special case but the usual case that I am
> want
> > it to work
> > as it should :). Lets think of a multi-module build or
> a
> > project that
> > has the pom.xml in a sub-directory of the project. I
> will
> > try to
> > explain more with 2 examples.
> > 
> > First, a single module project. Let us suppose that we
> have
> > a project
> > tree as follows:
> > 
> > root
> > |->project
> >      |->pom.xml
> > |->some_other_stuff
> > 
> > Now when maven will take changelog of it the history
> > rationally should
> > include only histories of the changes under 'project'
> > sub-tree and not
> > of all the project. Adding the '.' will do just that.
> > 
> > Secondly, let us take a multi-module project. In that
> case
> > when a
> > changelog is requested it should show the changes made
> to
> > that tree,
> > and not whole project. Adding the '.' will do just
> that.
> > 
> > > I'm not really sure if we can safely assume to
> always
> > execute in the modules path. To be honest, I doubt it!
> I
> > have something in my mind, but I'm not sure which case
> it
> > was. Maybe CI builds, hmmm? In those cases a single
> '.'
> > would not be sufficient.
> > >
> > 
> > I would be grateful if you would kindly state an
> example
> > where it
> > would not be sufficient with the change and was
> without
> > it.
> > 
> > > And there is a really subtle difference with git
> in
> > releasing multi-module builds.
> > >
> > 
> > Yes that is true, but I do not see that effecting
> release
> > process :).
> > 
> > > As you know, Git only handles full trees and not
> > single files. So the behaviour of a release differes
> if the
> > git repo contains all the modules (including parent)
> or if
> > there is a git-submodule involved.
> > >
> > 
> > What I am not sure how does it adversely effect
> release
> > behavior? It
> > is not clear to me.
> > 
> > > So there are most probably still a few things
> left to
> > do.
> > >
> > 
> > Please point them out and I will gladly help out. But
> this
> > is a change
> > I think would be beneficial to many not to mention
> that I
> > am eagerly
> > waiting to see its introduction :). Without this in a
> > multi-module
> > scenario changelog does not make much (if any at all)
> > sense.
> > 
> > Thank you,
> > 
> > > LieGrue,
> > > strub
> > >
> > > --- Imran M Yousuf <imyou...@gmail.com>
> > schrieb am Di, 7.4.2009:
> > >
> > >> Von: Imran M Yousuf <imyou...@gmail.com>
> > >> Betreff: [PATCH] Git Change log takes current
> path
> > into account
> > >> An: scm-dev@maven.apache.org
> > >> CC: "Mark Struberg" <strub...@yahoo.de>
> > >> Datum: Dienstag, 7. April 2009, 5:23
> > >> Hi,
> > >>
> > >> I was just checking the commands generated by
> git
> > changelog
> > >> and I
> > >> noticed that it does not respect the path,
> whereas
> > it is
> > >> very easy to
> > >> integrate it, mentioning a simple '.' at the
> end
> > of the
> > >> 'git
> > >> whatchanged' command does the trick and thats
> what
> > the
> > >> attached patch
> > >> does.
> > >>
> > >> I would be grateful if this would be
> integrated
> > with the
> > >> current
> > >> version as this change has a grave positive
> effect
> > on maven
> > >> site's
> > >> changelog report.
> > >>
> > >> Thank you,
> > >>
> > >> --
> > >> Imran M Yousuf
> > >> Entrepreneur & Software Engineer
> > >> Smart IT Engineering
> > >> Dhaka, Bangladesh
> > >> Email: im...@smartitengineering.com
> > >> Blog: http://imyousuf-tech.blogs.smartitengineering.com/
> > >> Mobile: +880-1711402557
> > >>
> > >
> > >
> > >
> > >
> > 
> > 
> > 
> > -- 
> > Imran M Yousuf
> > Entrepreneur & Software Engineer
> > Smart IT Engineering
> > Dhaka, Bangladesh
> > Email: im...@smartitengineering.com
> > Blog: http://imyousuf-tech.blogs.smartitengineering.com/
> > Mobile: +880-1711402557
> > 
> 
> 
> 
> 



Reply via email to