cnplab / cosmos

ClickOS toolstack
BSD 3-Clause "New" or "Revised" License
18 stars 16 forks source link

Click configuration file size #10

Closed bchatelard closed 10 years ago

bchatelard commented 10 years ago

I am trying to load a large generated click configuration using cosmos start $DOMID script.click, and i get an error saying that my file is too large.

Click configurations with more than 15000 bytes not supported.

In the code that the limit is hard coded: https://github.com/cnplab/cosmos/blob/39a14f690170ef62124cce17a2b97b40124f66ce/src/clickos.c#L186-L191

Why is there a file size limit ? Is there another way to load a large script ?

jpemartins commented 10 years ago

Hi,

Thanks for reporting and for trying things out! There is no particular reason to impose this limit. The hardcoded valur was an artifact of an older version of clickos/cosmos. But I removed it and is now fixed. I tested with a 20 KB config. Let us know if you have issues.

bchatelard commented 10 years ago

Ok, this works fine now, thanks !