search

jpvanhal / siilo

File storage abstraction layer for Python
http://siilo.readthedocs.org/
MIT License
23 stars 11 forks source link

Implemented CMIS integration #3

Open fmerges opened 10 years ago

fmerges commented 10 years ago

Hello,

I just wanted to let you know that I implemented a storage class for CMIS support, need to write the unit tests and then I will do a Pull-Request. And well then it's up to you to tweak it. ;-)

BTW, It's heavily based on ApacheLibcloudStorage, so it might be useful to have a more generic class so that both subclass from it. I think the strategy of working with a local temporary file is applicable for other storage classes as well... but I leave that for you to decide.

Cheers

PS: While writting it I discovered a bug with the current Cmislib 0.5.1, patch and issue are already posted.

jpvanhal commented 10 years ago

Hi Florian,

Great to hear from not only a user but a future contributor of Siilo!

Yep, LibcloudFile is pretty generic and I agree the temporary file strategy is applicable for other storages. I can refactor it to something that's reusable for other storages as well.

Looking forward to your pull request!