← Back to team overview

kicad-developers team mailing list archive

Re: Git transition

 

I'm OK with git-SUM rather than the obnoxiously long git rev numbers.
Anyone object to this?

On 8/12/2016 1:25 AM, Simon Wells wrote:
> In regards to this, Currently bzr revs are just the number and the git
> revs are the bzr number and the git sum. Is it worthwhile (esspeically
> in the title bar just changing this to git-BZRREV or just git-SUM
> since git will be the primary vcs?
> 
> On Fri, Aug 12, 2016 at 7:55 AM, Adam Wolf
> <adamwolf@xxxxxxxxxxxxxxxxxxxx> wrote:
>> Sounds good!  I think that timeline is fine.
>>
>> Adam Wolf
>>
>>
>> On Thu, Aug 11, 2016, 2:54 PM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>>
>>> You have a couple of options.
>>>
>>> You could just clone directly from the source mirror on github for
>>> testing then update the url when we go live with git on launchpad.
>>>
>>> You could create your own clone of the github source mirror and add a
>>> remote link to your own personal git branch on launchpad and use that
>>> for testing.  This is what I have been using.  The set the url to push
>>> to your own git repo on launchpad run:
>>>
>>> git remote add launchpad
>>> git+ssh://USER@xxxxxxxxxxxxxxxxx/~USER/kicad/+git/kicad-dev
>>>
>>> git push launchpad (OPTIONAL-REPO-NAME-ON-LP)
>>>
>>> You will now have your own kicad git repo on launchpad which you can
>>> clone for testing.
>>>
>>> On 8/11/2016 3:36 PM, Adam Wolf wrote:
>>>> It should.  Can we somehow push a copy of what it is now so I have
>>>> something to build against?
>>>>
>>>> On Thu, Aug 11, 2016, 1:52 PM Wayne Stambaugh <stambaughw@xxxxxxxxx
>>>> <mailto:stambaughw@xxxxxxxxx>> wrote:
>>>>
>>>>     If we go live with git on 8/21, will that give you enough time to
>>>> get
>>>>     things squared away on your end?
>>>>
>>>>     On 8/11/2016 2:41 PM, Adam Wolf wrote:
>>>>     > The Git part will take maybe 1 week for OS X.  I am in favor of
>>>> this
>>>>     > transition.
>>>>     >
>>>>     > Adam Wolf
>>>>     >
>>>>     >
>>>>     > On Aug 11, 2016 1:31 PM, "Wayne Stambaugh" <stambaughw@xxxxxxxxx
>>>>     <mailto:stambaughw@xxxxxxxxx>
>>>>     > <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>>
>>>> wrote:
>>>>     >
>>>>     >     On 8/11/2016 5:17 AM, Maciej Sumiński wrote:
>>>>     >     > On 08/10/2016 03:34 PM, Wayne Stambaugh wrote:
>>>>     >     >> On 8/10/2016 5:02 AM, Maciej Sumiński wrote:
>>>>     >     >>> On 08/08/2016 06:09 PM, Wayne Stambaugh wrote:
>>>>     >     >>>> The last time I looked, notifications of repo commits
>>>> still
>>>>     >     were not
>>>>     >     >>>> implemented.  This is a show stopper for me.  I don't
>>>> want to
>>>>     >     have to
>>>>     >     >>>> constantly grep the git commit log to see what changed.
>>>>     If change
>>>>     >     >>>> notifications are working correctly, then I'm OK with
>>>> moving
>>>>     >     forward on
>>>>     >     >>>> this if you can get the bug fix linking working.  We
>>>>     definitely
>>>>     >     should
>>>>     >     >>>> do some testing before we go live with this.
>>>>     >     >>>
>>>>     >     >>> I see there is an option to set notifications, in the same
>>>> way
>>>>     >     as for
>>>>     >     >>> the bazaar branches ("Edit your subscriptions" on the
>>>>     right side
>>>>     >     pane).
>>>>     >     >>> I could not verify it, as likely I cannot receive
>>>>     notifications
>>>>     >     for the
>>>>     >     >>> changes I introduce. Even if it does not work, I can
>>>> implement
>>>>     >     it in my
>>>>     >     >>> webhook.
>>>>     >     >>
>>>>     >     >> I spent some time yesterday creating my own git clone of
>>>>     kicad on
>>>>     >     LP and
>>>>     >     >> I noticed that the subscriptions that I need appear to be
>>>>     >     available for
>>>>     >     >> git repos so we shouldn't need any webhooks in for that
>>>> unless
>>>>     >     they do
>>>>     >     >> not work.
>>>>     >     >
>>>>     >     > If they do not work, let me know and I will fix it in the
>>>> hook.
>>>>     >     >
>>>>     >     >>>
>>>>     >     >>> The webhook has reached beta stage. I have created a dummy
>>>>     >     project for
>>>>     >     >>> testing purposes, where you can see a bug report [1] and a
>>>>     >     commit [2]
>>>>     >     >>> with message that includes a "fix(es)?[
>>>> ]+(lp:|#)?([0-9]+)"
>>>>     >     regex match.
>>>>     >     >>> When it is detected, it automatically adds a message,
>>>> changes
>>>>     >     the bug
>>>>     >     >>> status and assignee. One thing that is not possible right
>>>>     now is
>>>>     >     linking
>>>>     >     >>> with git branches, as apparently launchpad does not handle
>>>>     this
>>>>     >     at the
>>>>     >     >>> moment (or I could not find the right format to specify a
>>>>     branch).
>>>>     >     >>
>>>>     >     >> Bug report linking is very important to me since I am
>>>>     responsible for
>>>>     >     >> the stable branch.  If there is no support for this yet,
>>>>     I'm OK with
>>>>     >     >> adding the bug report number to the first line of the
>>>> commit
>>>>     >     message and
>>>>     >     >> the URL somewhere in the commit message body.  If I give
>>>> the OK
>>>>     >     to use
>>>>     >     >> git, I will expect all developers that have commit
>>>>     privileges to the
>>>>     >     >> product repo to follow this without exception.  The commit
>>>>     >     message for
>>>>     >     >> bug report fixes must have this format:
>>>>     >     >>
>>>>     >     >> Description of bug report fix. (fixes lp:########)
>>>>     >     >>
>>>>     >     >> * https://bugs.launchpad.net/kicad/+bug/########
>>>>     <https://bugs.launchpad.net/kicad/+bug/#%23%23%23%23%23%23%23>
>>>>     >     <https://bugs.launchpad.net/kicad/+bug/########
>>>>     <https://bugs.launchpad.net/kicad/+bug/#%23%23%23%23%23%23%23>>
>>>>     >     >>
>>>>     >     >> If this is not acceptable, then the git transition will
>>>>     have to wait
>>>>     >     >> until Canonical gets git bug report linking implemented or
>>>>     Orson
>>>>     >     beats
>>>>     >     >> them to it.
>>>>     >     >
>>>>     >     > I spoke with a Launchpad developer and they have it already
>>>>     in their
>>>>     >     > todo list. There is a plan to migrate Launchpad itself to
>>>>     git, so I
>>>>     >     > believe they will do it well.
>>>>     >     >
>>>>     >     > From what I heard, currently it is possible to link git
>>>>     merge requests
>>>>     >     > to bug reports, so it may temporarily solve the problem.
>>>>     >
>>>>     >     I'll see if I can figure out how to do this and if it works we
>>>>     can use
>>>>     >     it instead of adding the bug report url to the commit
>>>>     message.  I wonder
>>>>     >     if we can link a commit to a bug report?  That could be an
>>>>     issue if we
>>>>     >     cannot.  I don't want to have to always create a separate
>>>>     branch, push
>>>>     >     it to my personal repo, and then merge it into the product
>>>>     branch for
>>>>     >     simple bug fixes.
>>>>     >
>>>>     >     >
>>>>     >     >>> All we need to do is to set a webhook pointing to my
>>>>     script [3].
>>>>     >     If it
>>>>     >     >>> is accepted, then I am going to create a separate lp
>>>>     account for the
>>>>     >     >>> automated changes.
>>>>     >     >>>
>>>>     >     >>> Currently the webhook works on my home server which has a
>>>> high
>>>>     >     uptime,
>>>>     >     >>> but still it is not as reliable as dedicated servers. If
>>>>     there is
>>>>     >     >>> someone willing to host it on a better machine, I will be
>>>>     >     pleased to help.
>>>>     >     >>>
>>>>     >     >>> If you are curious about the source code, then I can put
>>>> it in
>>>>     >     the KiCad
>>>>     >     >>> github (once I get a repository there) or just post it
>>>>     somewhere.
>>>>     >     >>
>>>>     >     >> I can create a repo on github or you can create a repo on
>>>>     launchpad.
>>>>     >     >> Either way is fine by me.  If you want to use github, let
>>>>     me know
>>>>     >     what
>>>>     >     >> name you want for the repo and your github user name and I
>>>> will
>>>>     >     set up
>>>>     >     >> the repo and give you admin rights.
>>>>     >     >
>>>>     >     > I have just pushed the code to Launchpad [1] and consider it
>>>>     ready to
>>>>     >     > go. There is also a new account (KiCad Janitor) awaiting
>>>>     approval for
>>>>     >     > kicad-developers membership, so all the changes will be done
>>>>     using
>>>>     >     this
>>>>     >     > dedicated account.
>>>>     >     >
>>>>     >     > The webhook has been modified to accept a wider set of
>>>> phrases
>>>>     >     > indicating a bugfix (now it is (f|F)ix(es|ed|ing)?:?
>>>>     >     *(lp:|#)?([0-9]+)).
>>>>     >     >
>>>>     >     > Let me know when the git repository is set up, so I can
>>>> install
>>>>     >     the webhook.
>>>>     >
>>>>     >     This will require some coordination with our package devs.
>>>>     Package
>>>>     >     devs, when can we be ready to provide nightly builds from the
>>>>     git repo?
>>>>     >     Does anyone else have any issues with converting over to git?
>>>>     Speak now
>>>>     >     or forever hold your peace.
>>>>     >
>>>>     >     >
>>>>     >     > Regards,
>>>>     >     > Orson
>>>>     >     >
>>>>     >     > 1. https://launchpad.net/kicad-git-hook
>>>>     >     <https://launchpad.net/kicad-git-hook>
>>>>     >     >
>>>>     >     >> Thanks for working on this.
>>>>     >     >>
>>>>     >     >> Cheers,
>>>>     >     >>
>>>>     >     >> Wayne
>>>>     >     >>
>>>>     >     >>>
>>>>     >     >>> Regards,
>>>>     >     >>> Orson
>>>>     >     >>>
>>>>     >     >>> 1. https://bugs.launchpad.net/kicad-git-test/+bug/1611664
>>>>     >     <https://bugs.launchpad.net/kicad-git-test/+bug/1611664>
>>>>     >     >>> 2.
>>>>     >     >>>
>>>>     >
>>>>
>>>> https://git.launchpad.net/kicad-git-test/commit/?id=3d29b9be29346fdfaa87cdf8abf6957bf46bb5cd
>>>>     >
>>>>
>>>> <https://git.launchpad.net/kicad-git-test/commit/?id=3d29b9be29346fdfaa87cdf8abf6957bf46bb5cd>
>>>>     >     >>> 3. https://orson.net.pl/kicad_git_hook
>>>>     >     <https://orson.net.pl/kicad_git_hook>
>>>>     >     >>>
>>>>     >     >>>> Before every starts beating the GitHub drum, I have one
>>>> major
>>>>     >     issue with
>>>>     >     >>>> GitHub and that is control.  There is no way that I know
>>>>     of to
>>>>     >     moderate
>>>>     >     >>>> a project on github.  Anyone with a github account can
>>>>     submit a
>>>>     >     pull
>>>>     >     >>>> requests at anytime even if they are not part of the dev
>>>>     team.  As
>>>>     >     >>>> project leader, this is an issue.  I'm already a my limit
>>>>     with the
>>>>     >     >>>> development team we have in place and I really don't want
>>>> to
>>>>     >     deal with a
>>>>     >     >>>> wide open code hosting.  I also have no way of removing
>>>>     someone
>>>>     >     from the
>>>>     >     >>>> list should I need to.  I know it hasn't happened yet but
>>>>     I am
>>>>     >     not naive
>>>>     >     >>>> enough to think that it wont happen.  At this time, I am
>>>> more
>>>>     >     >>>> comfortable with LP until something better comes along or
>>>> we
>>>>     >     take full
>>>>     >     >>>> control a provide our own hosting.
>>>>     >     >>>>
>>>>     >     >>>> On 8/8/2016 3:58 AM, Maciej Sumiński wrote:
>>>>     >     >>>>> Apparently we have not discussed git transition for a
>>>> long
>>>>     >     time now, so
>>>>     >     >>>>> I felt it is the right time to raise the subject again
>>>>     to keep
>>>>     >     our routine.
>>>>     >     >>>>>
>>>>     >     >>>>> Launchpad now supports merge proposals [1] and it looks
>>>> like
>>>>     >     they are
>>>>     >     >>>>> still improving git integration. The only missing
>>>>     feature are
>>>>     >     links
>>>>     >     >>>>> between bug fixing commits and respective bug reports
>>>> (bzr
>>>>     >     commit ...
>>>>     >     >>>>> --fixes=lp:123456). It is not supported by git itself,
>>>> but
>>>>     >     might be
>>>>     >     >>>>> resolved using webhooks [2] and appropriate keywords in
>>>>     commit
>>>>     >     messages
>>>>     >     >>>>> (e.g. "Fixed a memory leak [fixes #123456]"). If this is
>>>>     the only
>>>>     >     >>>>> obstacle, then I volunteer to provide code for the hook.
>>>>     >     >>>>>
>>>>     >     >>>>> What do you think? Is there anything else that prevents
>>>>     >     transition?
>>>>     >     >>>>>
>>>>     >     >>>>> Regards,
>>>>     >     >>>>> Orson
>>>>     >     >>>>>
>>>>     >     >>>>> 1. https://help.launchpad.net/Code/Git
>>>>     >     <https://help.launchpad.net/Code/Git>
>>>>     >     >>>>> 2. https://help.launchpad.net/API/Webhooks
>>>>     >     <https://help.launchpad.net/API/Webhooks>
>>>>     >     >>>>>
>>>>     >     >>>>>
>>>>     >     >>>>>
>>>>     >     >>>>> _______________________________________________
>>>>     >     >>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     >>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>     >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>>>>     >     >>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     >>>>> More help   : https://help.launchpad.net/ListHelp
>>>>     >     <https://help.launchpad.net/ListHelp>
>>>>     >     >>>>>
>>>>     >     >>>>
>>>>     >     >>>> _______________________________________________
>>>>     >     >>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     >>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>     >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>>>>     >     >>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     >>>> More help   : https://help.launchpad.net/ListHelp
>>>>     >     <https://help.launchpad.net/ListHelp>
>>>>     >     >>>>
>>>>     >     >>>
>>>>     >     >>>
>>>>     >     >
>>>>     >     >
>>>>     >
>>>>     >     _______________________________________________
>>>>     >     Mailing list: https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>>>>     >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>>>>     >     Unsubscribe : https://launchpad.net/~kicad-developers
>>>>     >     <https://launchpad.net/~kicad-developers>
>>>>     >     More help   : https://help.launchpad.net/ListHelp
>>>>     >     <https://help.launchpad.net/ListHelp>
>>>>     >
>>>>
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>>


Follow ups

References