[Paraview-developers] Adding a check for local Git hooks

[David Cole]

Now that we've had much discussion about whether to have the hooks at all...
time for the next question.

If the hooks are changed in the master repository, how are those changes
going to be propagated?

The check for the hooks is just whether there exists a git repo in the
.git/hooks directory, not what it contains...

Any thoughts on how to make the hooks change-able so that when changes are
made to the master repo, they get propagated correctly? Or, at least, that I
get notified that my hooks are out of date and instructions on how to update
them manually....?


Thanks,
David C.



On Tue, Jul 6, 2010 at 10:13 AM, Marcus D. Hanwell <
marcus.hanwell at kitware.com> wrote:

> This is now in, I combined the VTK and ParaView local hook instructions
> when both are missing to avoid a double error. Please let me know if you
> have any problems getting the hooks up and running.
>
> Marcus
>
> On Mon, Jul 5, 2010 at 2:18 PM, Utkarsh Ayachit <
> utkarsh.ayachit at kitware.com> wrote:
>
>> Cool.
>>
>> Utkarsh
>>
>> On Mon, Jul 5, 2010 at 1:51 PM, Marcus D. Hanwell
>> <marcus.hanwell at kitware.com> wrote:
>> > No, I already handled that (the check is disabled if CTest is driving
>> the
>> > build). This is already the case with VTK as of this morning, and it is
>> > working well on the dashboards (but I will check the Nightly in the
>> > morning).
>> > Marcus
>> >
>> > On Mon, Jul 5, 2010 at 1:46 PM, Utkarsh Ayachit
>> > <utkarsh.ayachit at kitware.com> wrote:
>> >>
>> >> That sounds great. Of course, that would entail fixing all dashboard
>> >> machines :).
>> >>
>> >> Utkarsh
>> >>
>> >> On Mon, Jul 5, 2010 at 1:10 PM, Marcus D. Hanwell
>> >> <marcus.hanwell at kitware.com> wrote:
>> >> > Hi,
>> >> > Please see the discussion about checking if developers have the local
>> >> > git
>> >> > hooks installed on the VTK developer list,
>> >> > http://www.vtk.org/pipermail/vtk-developers/2010-June/008016.html
>> >> > I intend to add similar checks to ParaView, but wanted to make
>> ParaView
>> >> > developers aware of the change too. At configure time you will be
>> >> > prompted
>> >> > to install local git hooks if they are not installed. You can disable
>> >> > the
>> >> > check, and it is automatically disabled if you are not using a git
>> >> > clone, or
>> >> > if the build is being driven by CTest.
>> >> > This will ensure that all developers have hooks installed locally,
>> for
>> >> > more
>> >> > information about the hooks please see,
>> >> > http://www.vtk.org/Wiki/VTK/Git#Hooks
>> >> > Thanks,
>> >> > Marcus
>> >> > --
>> >> > Marcus D. Hanwell, Ph.D.
>> >> > R&D Engineer, Kitware Inc.
>> >> > (518) 881-4937
>> >> >
>> >> > _______________________________________________
>> >> > Paraview-developers mailing list
>> >> > Paraview-developers at paraview.org
>> >> > http://public.kitware.com/mailman/listinfo/paraview-developers
>> >> >
>> >> >
>> >
>> >
>>
>
>
> _______________________________________________
> Paraview-developers mailing list
> Paraview-developers at paraview.org
> http://public.kitware.com/mailman/listinfo/paraview-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview-developers/attachments/20100706/01103ee4/attachment.htm>