ontodev / valve.rs

A lightweight validation engine written in rust.
BSD 3-Clause "New" or "Revised" License
4 stars 0 forks source link

Weird problem with SQLite file path in export.py #30

Closed jamesaoverton closed 1 year ago

jamesaoverton commented 1 year ago

Running make sqlite_test I consistently get an error in export.py:

Traceback (most recent call last):
  File "/home/knocean/valve.rs/test/../scripts/export.py", line 397, in <module>
    with sqlite3.connect(db) as conn:
sqlite3.OperationalError: unable to open database file

The problem is this line https://github.com/ontodev/valve.rs/blob/main/scripts/export.py#L396. If I remove the "file:" part, it works fine. This doesn't make sense to me, since it's obviously working on @lmcmicu's machine and the GitHub Actions. My setup is:

@lmcmicu Is there any harm in changing that line to db = f"{path}{params}"?

lmcmicu commented 1 year ago

No there should be no harm, and I've tested it and it seems fine.

I'm not sure, but it's possible that this has something to do with the sqlite version. Mine is 3.34.1.