bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-06-02 Thread Ludovic Courtès
Hi Mathieu, Mathieu Othacehe skribis: >> I believe this is fixed by commit >> 20ed093977cc80ba1729c38e05ae7955a38069a6, which follows a modification >> to the ‘source-module-closure’ so that callers can catch >> missing-dependency errors. >> >> Please let me know what you think! > > It seems ok,

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-06-02 Thread Mathieu Othacehe
Hi Ludo, > I believe this is fixed by commit > 20ed093977cc80ba1729c38e05ae7955a38069a6, which follows a modification > to the ‘source-module-closure’ so that callers can catch > missing-dependency errors. > > Please let me know what you think! It seems ok, thanks for fixing it ! > > After that

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-25 Thread Ludovic Courtès
Mathieu Othacehe skribis: >> I'm having a hard time tring to understand ,trace output. > > I get why, it's because guile-ssh is not in my %load-path. So loading > "ssh/session.scm" fails in "source-module-dependencies". > > It's a bit of a vicious circle here : if resolve-interface '(ssh > sessio

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-19 Thread Ludovic Courtès
Mathieu Othacehe skribis: >> I'm having a hard time tring to understand ,trace output. > > I get why, it's because guile-ssh is not in my %load-path. So loading > "ssh/session.scm" fails in "source-module-dependencies". > > It's a bit of a vicious circle here : if resolve-interface '(ssh > sessio

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-19 Thread Mathieu Othacehe
> I'm having a hard time tring to understand ,trace output. I get why, it's because guile-ssh is not in my %load-path. So loading "ssh/session.scm" fails in "source-module-dependencies". It's a bit of a vicious circle here : if resolve-interface '(ssh session) fails, depends-on-guile-ssh? will f

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-19 Thread Mathieu Othacehe
Hi Ludo, It seems better but I can't get it to work neither : > (find (match-lambda > (('ssh _ ...) #t) > (_ #f)) > -(source-module-closure file #:select? (const #t > + (source-module-closure module #:select? (const #t)

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-19 Thread Ludovic Courtès
Mathieu Othacehe skribis: > * guix/build/pull.scm (depends-on-guile-ssh?): Remove. > (all-scheme-files): Use "scheme-modules" to detect all loadable > modules in given directory and rebuild a file list from those modules. > --- > guix/build/pull.scm | 31 --- > 1 file

bug#26987: [PATCH 2/2] build: pull: Fix compilation list construction.

2017-05-19 Thread Mathieu Othacehe
* guix/build/pull.scm (depends-on-guile-ssh?): Remove. (all-scheme-files): Use "scheme-modules" to detect all loadable modules in given directory and rebuild a file list from those modules. --- guix/build/pull.scm | 31 --- 1 file changed, 8 insertions(+), 23 deletions(