Page MenuHomeFreeBSD

New port: security/2fa: Two-factor authentication on the command line
ClosedPublic

Authored by yuri on Nov 24 2017, 5:26 AM.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

security/2fa/Makefile
19 ↗(On Diff #35678)

I don't reallly see the point of adding an option for a single readme file...

security/2fa/Makefile
19 ↗(On Diff #35678)

DOCS should control whether documentation is installed, whether it's one file or a hundred.

I think that at some point having options for options sake is not ideal, and that there is quite a middle ground between 1 and 100 :)

Now, if that file were to be generated via some depending port that uses tons of memory and time to build, I grant you it makes sense.
But for a static Readme file I think it's a bit much in general.

Otherwise the port is fine :)

This revision is now accepted and ready to land.Nov 24 2017, 6:04 PM
yuri marked 2 inline comments as done.Nov 24 2017, 6:59 PM
This revision was automatically updated to reflect the committed changes.