Skip to content

New website for TaurusTLS.#7

Open
JPeterMugaas wants to merge 1 commit into
tmssoftware:mainfrom
JPeterMugaas:taurustls-new-website
Open

New website for TaurusTLS.#7
JPeterMugaas wants to merge 1 commit into
tmssoftware:mainfrom
JPeterMugaas:taurustls-new-website

Conversation

@JPeterMugaas

Copy link
Copy Markdown
Contributor

No description provided.

@agallero

agallero commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Hi Peter,

This is something we should definitely look at, but right now we don't have a separate "website" field where you can write something like "url: https://taurustls.org/"

Right now, the "url" field is the one used by smartsetup to do a git clone of the repo. If I commit the changes here, when a user does:

tms install taurustls_developers.taurustls

SmartSetup would try to do a git clone from https://taurustls.org/ and I believe that will fail? (note that this doesn't matter in the tmsbuild.yaml you have in your repo, the url is only used in the registry to find where to clone. once we clone, we use the tmsbuild.yaml in the repo if it exists for all the other stuff)

The correct(™) solution here would be to have 2 fields, something like:

   repo url: https://github.com/TaurusTLS-Developers/TaurusTLS
   website url: https://taurustls.org/ 

But of course, this would break all existing stuff. This has grown organically, and field names aren't the best. So the alternatives I can propose are:

  1. We just add a "website" field where you can write this info:
   url: https://github.com/TaurusTLS-Developers/TaurusTLS
   website: https://taurustls.org/ 
  1. We deprecate url but keep supporting it (so we don't break all existing products), but for newer dev, we think in a better naming for those fields. As said, we could use "repo url/website url" or any other naming, I am open to any naming.

Any preferences here?

@JPeterMugaas

Copy link
Copy Markdown
Contributor Author

I think option 2 is the better option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants