bmx-routing / bmx7

BMX7 / SEMTOR Securely Entrusted Mesh Routing Protocol
GNU General Public License v2.0
63 stars 15 forks source link

bmx7-sms: increase max filename length to 32 #22

Closed aparcar closed 5 years ago

aparcar commented 6 years ago

32 characters should be enough for anyone I see no reason against longer filenames and it's especially useful when the filenames contain the shortid for remote command execution like here

axn commented 6 years ago

Haven't checked details yet but i am quite sure this patch will break compatibility between bmx7 versions using a 32b-sms plugin vs those using the old 16b version.

aparcar commented 6 years ago

Rather now then later I'd say. Is there any reason to keep it short?

aparcar commented 6 years ago

I've just checked and yes it does compatibility, even seems to break existing links not only not sharing the files.

aparcar commented 5 years ago

nevermind