Re: packaging Bootstrap for Cuirass' web interface

2018-07-29 Thread Clément Lassieur
Ludovic Courtès writes: > Hi! > > Clément Lassieur skribis: > >> I also get this if I send many requests at the same time. >> >> Uncaught exception in fiber ##f: >> In ice-9/boot-9.scm: >> 841:4 4 (with-throw-handler _ _ _) >> In web/server/fiberized.scm: >> 2018-07-25T10:33:56 HTTP connect

Re: packaging Bootstrap for Cuirass' web interface

2018-07-29 Thread Ludovic Courtès
Clément Lassieur skribis: > Clément Lassieur writes: > >>> That makes it clear that the reply channel is used only by the calling >>> fiber. >> >> I like it, but in that case MAKE-CHANNEL is called every time >> CALL-WITH-CRITICAL-SECTION is called. Do you think it is a significant >> overhead?

Re: packaging Bootstrap for Cuirass' web interface

2018-07-27 Thread Clément Lassieur
Clément Lassieur writes: >> That makes it clear that the reply channel is used only by the calling >> fiber. > > I like it, but in that case MAKE-CHANNEL is called every time > CALL-WITH-CRITICAL-SECTION is called. Do you think it is a significant > overhead? I guess it doesn't. I'll push a pa

Re: packaging Bootstrap for Cuirass' web interface

2018-07-27 Thread Clément Lassieur
Ludovic Courtès writes: > Or maybe like this: > > (define (make-critical-section . args) > "Return a channel used to implement a critical section. That channel can > then be passed to 'join-critical-section', which will ensure sequential > ordering. ARGS are the arguments of the critical sect

Re: packaging Bootstrap for Cuirass' web interface

2018-07-27 Thread Ludovic Courtès
Hi Clément! Clément Lassieur skribis: > The bug is here: > > (define (make-critical-section . args) > "Return a channel used to implement a critical section. That channel can > then be passed to 'join-critical-section', which will ensure sequential > ordering. ARGS are the arguments of the c

Re: packaging Bootstrap for Cuirass' web interface

2018-07-27 Thread Chris Marusich
Björn Höfling writes: > On Mon, 23 Jul 2018 09:48:15 +0200 > Clément Lassieur wrote: > >> Hello! >> >> We are going to embed Bootstrap files to our Cuirass service. >> Wouldn't it be better to package Bootstrap and get the service to use >> the package directly? I think it would make maintenan

Re: packaging Bootstrap for Cuirass' web interface

2018-07-26 Thread Clément Lassieur
Ludovic Courtès writes: > Clément Lassieur skribis: > >> Gábor Boskovits writes: >> >>> Hello Clément, >>> >>> Do you have any idea what has gone wrong here? >> >> Yes, I think Fibers channels aren't thread-safe. > > It’s always a possibility but that seems unlikely. Indeed, I spoke too soon :

Re: packaging Bootstrap for Cuirass' web interface

2018-07-26 Thread Ludovic Courtès
Hello Tatiana & Clément, Clément Lassieur skribis: > Tatiana Sholokhova writes: > >> Hi, Clement! >> >> Done! =) I’ve changed commit message with static files and rebased the >> latest commits, so my branch has only two commits now as we discuss >> before. > > Great! I rebased your commits ont

Re: packaging Bootstrap for Cuirass' web interface

2018-07-26 Thread Ludovic Courtès
Clément Lassieur skribis: > Gábor Boskovits writes: > >> Hello Clément, >> >> Do you have any idea what has gone wrong here? > > Yes, I think Fibers channels aren't thread-safe. It’s always a possibility but that seems unlikely. Ludo’.

Re: packaging Bootstrap for Cuirass' web interface

2018-07-26 Thread Ludovic Courtès
Hi! Clément Lassieur skribis: > I also get this if I send many requests at the same time. > > Uncaught exception in fiber ##f: > In ice-9/boot-9.scm: > 841:4 4 (with-throw-handler _ _ _) > In web/server/fiberized.scm: > 2018-07-25T10:33:56 HTTP connection from 127.0.0.1 >124:18 3 (_) >

Re: packaging Bootstrap for Cuirass' web interface

2018-07-26 Thread Clément Lassieur
Gábor Boskovits writes: > Hello Clément, > > Do you have any idea what has gone wrong here? Yes, I think Fibers channels aren't thread-safe. The bug happens when CALL-WITH-CRITICAL-SECTION is called at the same time from two different threads. So, nothing to do with Tatiana's code. :-) I'll i

Re: packaging Bootstrap for Cuirass' web interface

2018-07-25 Thread Clément Lassieur
Gábor Boskovits writes: >> > Hmm there seems to be a problem: >> > >> > 2018-07-25T10:08:48 GET / >> > In cuirass/http.scm: >> >292:21 3 (url-handler _ _ #< getq: #) >> > In cuirass/templates.scm: >> > 73:16 2 (specifications-table #) >> > In srfi/srfi-1.scm: >> > 589:5 1 (map # ?)

Re: packaging Bootstrap for Cuirass' web interface

2018-07-25 Thread Gábor Boskovits
Clément Lassieur ezt írta (időpont: 2018. júl. 25., Sze, 10:46): > Clément Lassieur writes: > > > Clément Lassieur writes: > > > >> Hi Tatiana, > >> > >> Tatiana Sholokhova writes: > >> > >>> Hi, Clement! > >>> > >>> Done! =) I’ve changed commit message with static files and rebased the > >>>

Re: packaging Bootstrap for Cuirass' web interface

2018-07-25 Thread Clément Lassieur
Hello Tatiana! An interesting command to test that everything works correctly is: parallel 'echo {} && curl http://localhost:8080/' ::: {1..100} You can use it on an example server: ./pre-inst-env cuirass --specifications=examples/random.scm --database=test.db or ./pre-inst-env cuirass --specif

Re: packaging Bootstrap for Cuirass' web interface

2018-07-25 Thread Clément Lassieur
Clément Lassieur writes: > Clément Lassieur writes: > >> Hi Tatiana, >> >> Tatiana Sholokhova writes: >> >>> Hi, Clement! >>> >>> Done! =) I’ve changed commit message with static files and rebased the >>> latest commits, so my branch has only two commits now as we discuss >>> before. >> >> Grea

Re: packaging Bootstrap for Cuirass' web interface

2018-07-25 Thread Clément Lassieur
Clément Lassieur writes: > Hi Tatiana, > > Tatiana Sholokhova writes: > >> Hi, Clement! >> >> Done! =) I’ve changed commit message with static files and rebased the >> latest commits, so my branch has only two commits now as we discuss >> before. > > Great! I rebased your commits onto my "multi

Re: packaging Bootstrap for Cuirass' web interface

2018-07-24 Thread Clément Lassieur
Hi Tatiana, Tatiana Sholokhova writes: > Hi, Clement! > > Done! =) I’ve changed commit message with static files and rebased the > latest commits, so my branch has only two commits now as we discuss > before. Great! I rebased your commits onto my "multiple inputs" changes. I couldn't force-pu

RE: packaging Bootstrap for Cuirass' web interface

2018-07-24 Thread Tatiana Sholokhova
: Gábor Boskovits; Guix-devel Тема: Re: packaging Bootstrap for Cuirass' web interface Ludovic Courtès writes: > Hi there! > > Gábor Boskovits skribis: > >> Clément Lassieur ezt írta (időpont: 2018. júl. 23., >> H, 9:48): >> >>> Hello! >>> >

Re: packaging Bootstrap for Cuirass' web interface

2018-07-23 Thread Clément Lassieur
Ludovic Courtès writes: > Hi there! > > Gábor Boskovits skribis: > >> Clément Lassieur ezt írta (időpont: 2018. júl. 23., >> H, 9:48): >> >>> Hello! >>> >>> We are going to embed Bootstrap files to our Cuirass service. Wouldn't >>> it be better to package Bootstrap and get the service to use t

Re: packaging Bootstrap for Cuirass' web interface

2018-07-23 Thread Clément Lassieur
Björn Höfling writes: > On Mon, 23 Jul 2018 09:48:15 +0200 > Clément Lassieur wrote: > >> Hello! >> >> We are going to embed Bootstrap files to our Cuirass service. >> Wouldn't it be better to package Bootstrap and get the service to use >> the package directly? I think it would make maintenan

Re: packaging Bootstrap for Cuirass' web interface

2018-07-23 Thread Björn Höfling
On Mon, 23 Jul 2018 09:48:15 +0200 Clément Lassieur wrote: > Hello! > > We are going to embed Bootstrap files to our Cuirass service. > Wouldn't it be better to package Bootstrap and get the service to use > the package directly? I think it would make maintenance easier, and > I'm unsure it's g

Re: packaging Bootstrap for Cuirass' web interface

2018-07-23 Thread Ludovic Courtès
Hi there! Gábor Boskovits skribis: > Clément Lassieur ezt írta (időpont: 2018. júl. 23., > H, 9:48): > >> Hello! >> >> We are going to embed Bootstrap files to our Cuirass service. Wouldn't >> it be better to package Bootstrap and get the service to use the package >> directly? I think it wou

Re: packaging Bootstrap for Cuirass' web interface

2018-07-23 Thread Gábor Boskovits
Clément Lassieur ezt írta (időpont: 2018. júl. 23., H, 9:48): > Hello! > > We are going to embed Bootstrap files to our Cuirass service. Wouldn't > it be better to package Bootstrap and get the service to use the package > directly? I think it would make maintenance easier, and I'm unsure it's