a1b10 / cl-xlsx

📜 Read XLSX files with Common Lisp
23 stars 2 forks source link

cl-xlsx: fix dependencies #14

Closed ebzzry closed 3 years ago

ebzzry commented 3 years ago

Hi! Loading cl-xlsx via Quicklisp yields this:

[package cl-xlsx].;
; caught ERROR:
;   READ error during COMPILE-FILE:
;
;     Package FXML.STP does not exist.

This patch (somehow) fixes that. I based the name of the dependency from ruricolist/FXML. Is that right?

gwangjinkim commented 3 years ago

Hey, so nice to hear from you! I couldn't check yet. The thing was it was fixed by slyrus - the fix introduced the dependency from FXML. I see the necessity to write some tests ... Thank you so much for the fix! I'll merge is as soon as I got into it - but it can take some time.

ebzzry commented 3 years ago

Thank you! I’m looking forward to it!

gwangjinkim commented 3 years ago

Sorry for confirming so late!

ebzzry commented 3 years ago

@gwangjinkim Thank you!