namecoin / StemNS

Implement Proposition 279 (Pluggable Naming) w/o changing Tor (Stem port)
https://www.namecoin.org/
The Unlicense
7 stars 9 forks source link

Consider renaming poc.py #18

Closed JeremyRand closed 3 years ago

JeremyRand commented 3 years ago

The executable name poc.py is awful. It's legacy inherited from TorNS, and never should have entered production. I suspect that packagers will want it renamed. Perhaps stemns.py? Not sure what Debian's policies on capitalization are.

JeremyRand commented 3 years ago

@rllola What's your opinion?

rllola commented 3 years ago

Renaming it to stemns.py sounds good. I am not aware on restriction about capitalization but I can check.

JeremyRand commented 3 years ago

I can't find any formal Debian policy on capitalization of executable script filenames, but it looks like capital letters are allowed but extremely rare. (ZuluCrypt uses a capital letter in its filename.) Lowercase sounds good to me.

Also, as per http://5ekxbftvqg26oir5wle3p27ax3wksbxcecnm6oemju7bjra2pn26s3qd.onion/doc/debian-policy/ch-files.html#scripts :

When scripts are installed into a directory in the system PATH, the script name should not include an extension such as .sh or .pl that denotes the scripting language currently used to implement it.

So we'll need to remove the .py extension when installing as a GNU/Linux package. The .py extension is still required on Windows, so I'll leave it present in the Git repo; the package will therefore need to rename it when installing to /usr/bin.