Skip to content

Commit 5971b8b

Browse files
committed
salmonella: better support for CHICKEN 5 units
Some units that are shipped with CHICKEN 4 are not shipped with CHICKEN 5 (e.g., srfi-1, srfi-13, srfi-18 ...), so don't warn when they are specified as dependencies in .meta.
1 parent 6e224ed commit 5971b8b

File tree

1 file changed

+16
-7
lines changed

1 file changed

+16
-7
lines changed

salmonella.scm

+16-7
Original file line numberDiff line numberDiff line change
@@ -92,11 +92,16 @@
9292
(pp (report->list report)))
9393
append:))
9494

95-
(define (chicken-unit? lib)
96-
(and (memq lib '(library eval expand data-structures ports files
97-
extras irregex srfi-1 srfi-4 srfi-13 srfi-14
98-
srfi-18 srfi-69 posix utils tcp lolevel foreign))
99-
#t))
95+
(define (chicken-unit? lib major-version)
96+
(and
97+
(memq lib
98+
(if (< major-version 5)
99+
'(library eval expand data-structures ports files
100+
extras irregex srfi-1 srfi-4 srfi-13 srfi-14
101+
srfi-18 srfi-69 posix utils tcp lolevel foreign)
102+
'(library eval expand data-structures ports files
103+
extras irregex srfi-4 posix utils tcp lolevel foreign)))
104+
#t))
100105

101106
(define (egg-installed? egg repo-lib-dir)
102107
(let ((installed-eggs
@@ -310,7 +315,9 @@
310315
(directory-exists? ;; workaround for issue with chicken 4.5.0 and regex
311316
(make-pathname tmp-dir (->string egg))))
312317
(install-egg egg 'install-test-dep)))))
313-
(remove chicken-unit? (or test-deps '()))))
318+
(remove (lambda (dep)
319+
(chicken-unit? dep major-version))
320+
(or test-deps '()))))
314321
(let ((test-dir (make-pathname (if this-egg?
315322
#f
316323
(list tmp-dir (->string egg)))
@@ -435,7 +442,9 @@
435442
(define (check-dependencies egg meta-data)
436443
(let* ((egg-deps (get-egg-dependencies meta-data 'with-test-deps))
437444
(invalid-deps
438-
(filter chicken-unit? egg-deps))
445+
(filter (lambda (dep)
446+
(chicken-unit? dep major-version))
447+
egg-deps))
439448
(invalid-deps? (not (null? invalid-deps)))
440449
(non-symbol-deps (remove (lambda (egg) (symbol? egg)) egg-deps)))
441450
(make-report egg 'check-dependencies (not (or invalid-deps? (not (null? non-symbol-deps))))

0 commit comments

Comments
 (0)