[Kde-scm-interest] New life in kdesdk-migration-to-git :) (was: Re: Layout of "lokalize" and "kompare" git repos with plugins integrated (was: Re: Forming repos by plugin type or code domain?))

[Jeremy Whiting]

Ok, today I found out why all the tag annotations were missing in
kde-dev-utils (and a few in dolphin-plugins also)  The attached patch fixes
the issue, the problem is that if a branch is created with nothing at the
top level, svn2git doesn't create the branch, and if there's no branch it
doesn't create the annotated tag, etc.  The attached patch to svn2git fixes
the issue and the fixed dolphin-plugins and kde-dev-utils are on
scratch/whiting/dolphin-plugins.git and kde-dev-utils.git

It seems when torgny added the path.isEmpty() he was fixing deleting
branches when only a subfolder gets removed.  This path.isEmpty() is where
it's creating branches though, not deleting them.  I can push this commit
to svn2git master if it looks ok with you guys.

thanks,
Jeremy


On Sat, Dec 29, 2012 at 9:39 AM, Jeremy Whiting <jpwhiting at kde.org> wrote:

> Ok, I added one.  It seems to build and install fine here.  Though I did
> notice kde-dev-utils has both doc/kmtrace and kmtrace/doc with the same
> contents  I'll see if I can find out why and not add the one in doc/kmtrace.
>
> thanks,
> Jeremy
>
>
> On Sat, Dec 29, 2012 at 9:05 AM, Albert Astals Cid <aacid at kde.org> wrote:
>
>> El Dissabte, 29 de desembre de 2012, a les 12:57:27, Sebastian Dörner va
>> escriure:
>> > On 29 December 2012 00:47, Albert Astals Cid <aacid at kde.org> wrote:
>> > > El Divendres, 28 de desembre de 2012, a les 13:36:39, Jeremy Whiting
>> va
>> > >
>> > > escriure:
>> > > > I've also just pushed new repos to scratch/whiting using the latest
>> > > > rules
>> > > > and parentmaps.  There is one issue that some of the tags aren't
>> > >
>> > > annotated
>> > >
>> > > > so are failing the hook to be pushed, but otherwise check them out
>> and
>> > >
>> > > see
>> > >
>> > > > if anything is missing.  I'll look into the tag issue.
>> > >
>> > > CMake Error: The source directory "kde-dev-utils" does not appear to
>> > > contain
>> > > CMakeLists.txt.
>> >
>> > This is because there is none. This repo is entirely new in its
>> collection
>> > of utils, so we don't have a CMakeLists.txt. I would therefore argue to
>> add
>> > one after the conversion.
>>
>> Can we get that in then? Without it it is quite hard to know if
>> installing the
>> git repos gives me the same files that installing the svn repo since i
>> can't
>> install kde-dev-utils.
>>
>> Cheers,
>>   Albert
>>
>> >
>> > > strigi-analyzers seems a weak name, it ought to be something like
>> > > kdesdk-string-analyzers, since the repo name will be kde:xxx and you
>> don't
>> > > want to "own" kde:strigi-analyzers
>> > >
>> > > Same for thumbnailers
>> >
>> > I just adapted the rules to use the names kdesdk-strigi-analyzers and
>> > kdesdk-thumbnailers.
>> >
>> > > the umbrello repo is huge, anything we could do so it's not that big?
>> > >
>> > > sandsmark in git says "git gc --aggressive && git prune gets it down
>> to
>> > > 28mb
>> > > for me" maybe we can do that to all the repos?
>> >
>> > As we usually did that in previous conversions, I agree we should do it
>> > here.
>> >
>> > > Cheers,
>> > >
>> > >   Albert
>> > >
>> > > > thanks,
>> > > > Jeremy
>> > > >
>> > > >
>> > > > On Fri, Dec 28, 2012 at 8:23 AM, Sebastian Dörner <
>> > > >
>> > > > sebastian at sebastian-doerner.de> wrote:
>> > > > > I've been working with Jeremy (and Nicolas) so we can finally get
>> this
>> > > > > over with. This is a heads-up about the strigi-analyzers. The
>> current
>> > > > > situation is as follows:
>> > > > >
>> > > > > 1)  Nobody seems to have very strong opinions about whether to
>> have
>> > > > > the
>> > > > > analyzers with the apps or their own repo (among the stakeholders
>> > > > > Nick,
>> > > > > Kevin, Friedrich, Jeremy, me)
>> > > > > 2)  Nick prefers the analyzers separate, because the analyzers
>> don't
>> > > > > depend on lokalize.
>> > > > > 3)  Kevin is unsure, would like to be consistent with whatever
>> > > > > Lokalize
>> > > > > does.
>> > > > > 4) KDE / Nepomuk will probably move away from strigi in the
>> future.
>> > > > > 5) Currently distributions often have a separate package
>> > > > > kdesdk-strigi-analyzers (or similar).
>> > > > > 6) It is easiest from a rule-writing pov to keep the analyzers
>> > >
>> > > separate.
>> > >
>> > > > > Particularly because of points 4 and 5 we are currently
>> implementing a
>> > > > > solution with separate repos for strigi-analyzers (all of them)
>> and
>> > > > > thumbnailers. Neither the kompare nor the lokalize repo would then
>> > >
>> > > contain
>> > >
>> > > > > any strigi-analyzers.
>> > > > > If you have any objections, please bring them forward now.
>> > > > >
>> > > > > Regards
>> > > > > Sebastian
>> > > > >
>> > > > > _______________________________________________
>> > > > > Kde-scm-interest mailing list
>> > > > > Kde-scm-interest at kde.org
>> > > > > https://mail.kde.org/mailman/listinfo/kde-scm-interest
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-scm-interest/attachments/20121231/0bdcf5ec/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixbranches.diff
Type: application/octet-stream
Size: 610 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-scm-interest/attachments/20121231/0bdcf5ec/attachment.obj>