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

spectacle name conflict with KDE spectacle screenshot tool in Fedora repos #39

Closed
jn64 opened this issue Jun 12, 2023 · 8 comments · Fixed by #42
Closed

spectacle name conflict with KDE spectacle screenshot tool in Fedora repos #39

jn64 opened this issue Jun 12, 2023 · 8 comments · Fixed by #42
Assignees

Comments

@jn64
Copy link
Collaborator

jn64 commented Jun 12, 2023

Audinux spectacle package (spectrum analyzer) conflicts in name with Fedora spectacle package (KDE screenshot tool)

I think our package must be renamed.

Problem

  • Since Audinux package has a lower version, it cannot be installed by dnf install spectacle.
  • Even if you specify version/repo to get the Audinux package, dnf is confused by the metadata of both packages, and will install Fedora package's dependencies (qt/kde packages).
@jn64 jn64 converted this from a draft issue Jun 12, 2023
@ycollet
Copy link
Collaborator

ycollet commented Jun 12, 2023

Thanks a lot for this check !

@jn64
Copy link
Collaborator Author

jn64 commented Jun 13, 2023

Upstream already chose the name spectacle-analyzer because of this conflict. We should rename the package to that.

(I'm not sure of the process, if you need to take care of anything in the scripts or website)

@ycollet
Copy link
Collaborator

ycollet commented Jun 13, 2023

I think this is managed via the Obsolete: tag in the spec. But this is "risky" because there is some kind of overlap with the KDE package.
Maybe the best is to just rename the package and I will put a warning on the website. I personaly think we should do that.

@jn64
Copy link
Collaborator Author

jn64 commented Jun 13, 2023

Yes, we can't use Obsolete: here.

@ycollet
Copy link
Collaborator

ycollet commented Jun 13, 2023

Yes, but we must be really careful not to obsolete the spectacle KDE package :)
That's why I think it's better not to use obsolete and just add a message on the website.

@jn64
Copy link
Collaborator Author

jn64 commented Jun 14, 2023

It seems like dnf was not confused, but I am...

Somehow, the Audinux package on Copr is actually the KDE spectacle. Look at the spec file or build log of the last build: https://copr.fedorainfracloud.org/coprs/ycollet/audinux/build/5678346/

@jn64 jn64 self-assigned this Jun 14, 2023
@ycollet
Copy link
Collaborator

ycollet commented Jun 14, 2023

Hmm; excellent. I really don't know what happened here ???
I will remove ASAP all builds of spectable in the repo. I will wait for the renaming of the spec are rebuild spectacla after that.

@ycollet
Copy link
Collaborator

ycollet commented Jun 14, 2023

Just checked and removed all the builds of spectacle.

@jn64 jn64 moved this from Todo to In Progress in @jn64's TODO Jun 14, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in @jn64's TODO Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants