Closed strongX509 closed 10 years ago
Looks good to me. @dbrgn can you please check the installation instructions?
I think there's a typo before the last listing:
which currently consists of list of all files installed by the package
On a side note: we're trying to remove the yum-utils dependency by only using the rpm command. We keep you posted about this.
Regards Christian
We can remove the yum and yum-utils dependencies at all and use the rpm tool directly, since yum is only an frontend for the rpm database (see https://github.com/tnc-ba/strongTNC/issues/108) Implemented in https://github.com/tnc-ba/swidGenerator/pull/17
We've merged https://github.com/tnc-ba/swidGenerator/pull/17 into the master branch. You can remove yum-utils from the HOWTO in the strongSwan wiki.
Regards Christian
On a related issue (as you're using the name in the HowTo), @strongX509, do you have a preference on the official name of the program?
The "binary" is called swid_generator
, as Python packages are not supposed to use CamelCase.
For the "verbose" name of the program we've mostly used "SWID Generator" so far. If you prefer we can use swidGenerator
, so it fits better with strongSwan and strongTNC. What do you think?
Besides that, the installation instructions look good. We'll probably upload the tool to the Python Package Index soon (https://github.com/tnc-ba/strongTNC/issues/107), after that it can be installed directly via pip install swid_generator
, without having to download it manually first.
swid_generator as the name of the binary is ok. swidGenerator as the name of the project is fine with me.
I wrote a swidGenerator HOWTO on the strongSwan wiki:
http://wiki.strongswan.org/projects/strongswan/wiki/SwidGenerator
I hope I got everything right.
Cheers
Andreas