On Thu, Jan 9, 2014 at 5:58 PM, Mark Janssen <mpc.jans...@gmail.com> wrote:

>
>
>
> On Thu, Jan 9, 2014 at 5:31 PM, Jan Nijtmans <jan.nijtm...@gmail.com>wrote:
>
>> 2014/1/9 Jan Nijtmans <jan.nijtm...@gmail.com>:
>> > I have a different fix
>> > in mind, I'll come back on that later.
>>
>> <http://fossil-scm.org/index.html/timeline?r=delay-ticket-hook>
>>
>> Does this work for you?
>>
>> Regards,
>>      Jan Nijtmans
>> _______________________________________________
>> fossil-users mailing list
>> fossil-users@lists.fossil-scm.org
>> http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users
>>
>
>
> No with this a ticket change from the web UI will not trigger the xfer
> script.
>


With change below it works again is rc what you think it is at that part in
the code? I would think that a single ticket hook script failure should not
terminate all of them.

--- src/manifest.c
+++ src/manifest.c
@@ -1506,13 +1506,11 @@
   }
   db_prepare(&q, "SELECT uuid FROM pending_tkt");
   while( db_step(&q)==SQLITE_ROW ){
     const char *zUuid = db_column_text(&q, 0);
     ticket_rebuild_entry(zUuid);
-    if( rc==TH_OK ){
-      rc = xfer_run_script(xfer_ticket_code(), zUuid);
-    }
+    rc = xfer_run_script(xfer_ticket_code(), zUuid);
   }
   db_finalize(&q);
   db_multi_exec("DROP TABLE pending_tkt");

   /* If multiple check-ins happen close together in time, adjust their
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to