Skip to content
This repository has been archived by the owner on Nov 16, 2021. It is now read-only.

Develop #7

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Develop #7

wants to merge 10 commits into from

Conversation

ABartalesi
Copy link

Removed automatic conversion from .ui to .py at install time.
Removed automatic TAURUSQTDESIGNERPATH script creation.
Made compatible with gitlab-CI.

@ABartalesi
Copy link
Author

It's a bit strange that the python setup creates a file under /etc/profile.d/
It will need higher credential, a user launched pip install will not work.
However, this problem was there already, it's just not fixed (I don't know a better way, since environment variables need to be set somehow).

Now it passes the pipeline.

@hardion
Copy link
Member

hardion commented May 3, 2021

This is probably related how the rpm package is made. setup.py is maybe not the best place but deployed with rpm, a environment setup script in /etc/profile.d make lot of sense.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants