Hi, Danny, Thanks for comments!
Danny Milosavljevic <dan...@scratchpost.org> writes: > Hi, > > On Mon, 30 Jan 2017 11:06:49 +0800 > huang_ying_cari...@163.com wrote: > >> -(define* (manifest-lookup-package manifest name #:optional version) >> +(define* (manifest-lookup-package manifest name #:optional version output) >> "Return as a monadic value the first package or store path referenced by >> MANIFEST that is named NAME and optionally has the given VERSION prefix, or >> #f >> if not found." > > Please update the docstring too. Sure. >> + (if output >> + (string-suffix? output version*) >> #t)))) > > Should that include the separator between the parts? Otherwise it > could maybe have false positives... OK. Best Regards, Huang, Ying