Efraim Flashner writes:

> You also have to add the patch file to gnu-system.am so it gets picked
> up.

Thanks, 

>> +   (native-search-paths
> this line should line up with the others like (native-inputs. It looks like
> it also threw off (arguments

Reindented.

>> +    (list (search-path-specification
>> +           (variable "TCLLIBPATH")
>> +	   (separator " ")
>> +	   (files (list (string-append "lib/Tclxml" version))))))
> I don't think this needs to be a list. Also, (variable, (separator and (file
> should line up with the "e" in (search

> the two following string-appends should line up with the above
> string-append

Somehow TABs got in, fixed.

>> +      #:phases (modify-phases %standard-phases
>> +                 (delete 'check))))
> normally we put #:tests? #f, with a reason why. Are there no tests? Do they
> fail unexpectedly?

Ah, there is a test target that passes.  Using that now.

>> +    (home-page "http://tclxml.sourceforge.net/")
>> +    (synopsis " Tcl library for XML parsing")
>                   ^
>        extra space
>> +    (description " TclXML provides event-based parsing of XML documents.  The
>                      ^
>         extra space

Reindented.

> make sure you run guix lint, there are several spaces where you use
> tabs and we only use spaces

Ah, that helps!  Find new patch attached.

Greetings, Jan