Bug 1213893

Summary: [duplicate_package] duplicated chromedriver package from chromium and ungoogled-chromium
Product: [openSUSE] openSUSE Tumbleweed Reporter: Max Lin <mlin>
Component: OtherAssignee: Michał Szczepaniak <m.szczepaniak.000>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Normal    
Priority: P5 - None CC: Andreas.Stieger, gmbr3, guillaume.gardet, m.szczepaniak.000
Version: Current   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Max Lin 2023-08-02 07:57:28 UTC
chromedriver binary package has been found from two different package: chromium and ungoogled-chromium, this is something we're trying to avoid to have such duplicate package, please rename one of them, I'd suggest rename ungoogled-chromium's chromedriver to ungoogled-chromedriver, align with other binary package s ungoogled-chromium produced.
Comment 1 Max Lin 2023-08-02 07:57:53 UTC
what we found in Factory

x86_64:
  chromedriver:
  - chromium
  - ungoogled-chromium
Comment 2 Andreas Stieger 2023-08-02 08:15:02 UTC
Max you you couldn't look up the maintainer?
Michał can you please add a flavor name to the subpackage?
Comment 3 Michał Szczepaniak 2023-08-02 13:28:33 UTC
sure thing i'm on it
Comment 4 Callum Farmer 2023-08-02 13:32:51 UTC
(In reply to Max Lin from comment #0)
> chromedriver binary package has been found from two different package:
> chromium and ungoogled-chromium, this is something we're trying to avoid to
> have such duplicate package, please rename one of them, I'd suggest rename
> ungoogled-chromium's chromedriver to ungoogled-chromedriver, align with
> other binary package s ungoogled-chromium produced.

The amount of times I fixed this and then it got reverted by the package maintainer somehow is unbelievable 😑
Comment 5 Michał Szczepaniak 2023-08-02 13:39:02 UTC
I've not reverted anything, and if I did then i wasn't aware of that. 

https://build.opensuse.org/package/rdiff/network:chromium/ungoogled-chromium?linkrev=base&rev=8
this change is still included
https://build.opensuse.org/package/rdiff/network:chromium/ungoogled-chromium?linkrev=base&rev=14
so is this one

i can only see this one not included
https://build.opensuse.org/package/rdiff/network:chromium/ungoogled-chromium?linkrev=base&rev=6
becaues I misunderstood and thought the -n should be added

that's still just one time
Comment 6 Andreas Stieger 2023-08-02 13:51:30 UTC
Let's assume good intentions and just get it fixed.
Comment 7 Michał Szczepaniak 2023-08-02 13:52:33 UTC
Already building it in home branch will submit changes as soon as i confirm issue resolved
Comment 8 Michał Szczepaniak 2023-08-02 14:29:24 UTC
I've committed change that Callum made in past

https://build.opensuse.org/package/rdiff/network:chromium/ungoogled-chromium?linkrev=base&rev=35

And I added conflicts/provides from the beta version. 

https://build.opensuse.org/package/rdiff/network:chromium/ungoogled-chromium?linkrev=base&rev=36

After it builds i will submit it to factory. 

It creates package ungoogled-chromium-chromedriver which conflicts chromedriver, that should resolve issue
Comment 9 Michał Szczepaniak 2023-08-03 00:07:47 UTC
https://build.opensuse.org/request/show/1102076 request pending, i also made update while i was at it
Comment 10 Andreas Stieger 2023-08-05 11:35:58 UTC
$ osc ls -b openSUSE:Factory/ungoogled-chromium standard x86_64
_buildenv
_statistics
rpmlint.log
ungoogled-chromium-115.0.5790.102-1.1.src.rpm
ungoogled-chromium-115.0.5790.102-1.1.x86_64.rpm
ungoogled-chromium-chromedriver-115.0.5790.102-1.1.x86_64.rpm
ungoogled-chromium-chromedriver-debuginfo-115.0.5790.102-1.1.x86_64.rpm
ungoogled-chromium-debuginfo-115.0.5790.102-1.1.x86_64.rpm
Comment 11 Michał Szczepaniak 2023-08-05 11:38:15 UTC
and this

$ rpm -q --conflicts ./ungoogled-chromium-chromedriver-115.0.5790.98-20.1.x86_64.rpm 
chromedriver