[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug-gnu-arch] [bug #5690] Add a precommit hook

From: nobody
Subject: [Bug-gnu-arch] [bug #5690] Add a precommit hook
Date: Fri, 07 Nov 2003 20:44:07 -0500
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5a) Gecko/20031010 Mozilla Firebird/0.6.1 StumbleUpon/1.76

=================== BUG #5690: LATEST MODIFICATIONS ==================

Changes by: Robert Collins <address@hidden>
Date: Sat 11/08/2003 at 01:44 (GMT)

------------------ Additional Follow-up Comments ----------------------------
I've committed this into my archive as address@hidden/tla--devo--1.1--patch-22. 

=================== BUG #5690: FULL BUG SNAPSHOT ===================

Submitted by: mega-nerd               Project: GNU arch -- a revision control 
Submitted on: Thu 10/02/2003 at 10:55
Category:  tla                        Severity:  5 - Major                  
Bug Group:  small feature idea        Resolution:  None                     
Status:  Open                         Release:  tla-1.1pre5                 
Fixed Release:                        Merge Request?:  None                 
Your Archive Name:                    Your Archive Location:                
Assigned to:  None                    

Summary:  Add a precommit hook

Original Submission:  There is currently a hook in the commit function which 

at the end of the commit process. It would also be useful 

to have a hook run at the start of the commit process and 

be able to abort the commit process by returning a non-zero

error code from the hook script.

An example of what can be done with this is to do a make in 

the the hook script and have the script return make's error

code. This means that a compile will be done during the 

commit hook and if the code doesn't build it won't commit

and broken code doesn't get into the archive.

Obviously if someone wants to bypass this process they can.

This is meant to prevent accidental commits of broken code.

I have included a diff which seems to work for me, but I'm

not totally sure that its correct. I have also mentioned 

the previous version of this patch on the mailing list.

Follow-up Comments

Date: Sat 11/08/2003 at 01:44       By: robertc
I've committed this into my archive as address@hidden/tla--devo--1.1--patch-22. 

CC List

CC Address                          | Comment
robertc                             | 

File Attachments

Date: Thu 10/02/2003 at 10:55  Name: tla-1.1pre5-precommit.diff  Size: 1KB   
By: mega-nerd
A patch that seems to work.

For detailed info, follow this link:

  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]