Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove tito configuration #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lubomir
Copy link
Contributor

@lubomir lubomir commented Nov 24, 2020

It doesn't work to build a fully functioning package. In order to work, additional configuration is needed.

Right now the configuration file is stored in dist-git and used when building the Red Hat internal version of this package.

If tito should work, then one of these is needed:

  • the configuration has to be in this upstream repo (but it's specific for particular deployment of the package)
  • the configuration file has to move into a separate package (but ensuring it's installed is not trivial)
  • require users to get the configuration somewhere else

It doesn't work to build a fully functioning package. In order to work,
additional configuration is needed.

Right now the configuration file is stored in dist-git and used when
building the Red Hat internal version of this package.

If tito should work, then one of these is needed:
 * the configuration has to be in this upstream repo (but it's specific
   for particular deployment of the package)
 * the configuration file has to move into a separate package (but
   ensuring it's installed is not trivial)
 * require users to get the configuration somewhere else
@obriencj
Copy link
Contributor

Could you expand on what you mean that it doesn't work? I used this tito config to produce python-rcm-nexus builds successfully just a few days ago.

@lubomir
Copy link
Contributor Author

lubomir commented Nov 24, 2020

It will produce an RPM, but if you install it, you will be missing configuration file /etc/xdg/rcm-nexus.conf that tells it which Nexus instance to use, how the releases are promoted etc.
New users would need to get that from somewhere.

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