Bug 29963

Summary: The package works like a bpm analyzer
Product: Mageia Reporter: Nikolay Sabelnikov <79625490833>
Component: New RPM package requestAssignee: Mageia Bug Squad <bugsquad>
Status: RESOLVED INVALID QA Contact:
Severity: normal    
Priority: Normal CC: 79625490833, fri
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
URL: http://www.pogo.org.uk/~mark/bpm-tools/
Whiteboard:
Source RPM: bpm-tools CVE:
Status comment:
Bug Depends on: 29964    
Bug Blocks: 29464    
Attachments: Built-in BPM detection

Description Nikolay Sabelnikov 2022-01-30 12:31:14 CET
This package comes as a dependency to the mixxx package
Nikolay Sabelnikov 2022-01-30 12:31:58 CET

Blocks: (none) => 29464

Nikolay Sabelnikov 2022-01-30 12:32:49 CET

Blocks: 29464 => (none)
Version: Cauldron => 8

Nikolay Sabelnikov 2022-01-30 12:33:05 CET

Blocks: (none) => 29464

Nikolay Sabelnikov 2022-01-30 12:33:15 CET

CC: (none) => 79625490833

Nikolay Sabelnikov 2022-01-30 12:33:29 CET

Version: 8 => Cauldron

Nikolay Sabelnikov 2022-01-30 12:37:55 CET

Depends on: (none) => 29964

Comment 1 sturmvogel 2022-01-30 14:07:23 CET
How comes that a new release of mixxx would depend on an umaintained 9 years old tool when the prior versions didn't?

Also doublechecked on a Fedora 35 build: bpm-tools is not a dependency for building or running of mixxx.
Comment 2 Nikolay Sabelnikov 2022-01-30 14:24:51 CET
(In reply to sturmvogel from comment #1)
> How comes that a new release of mixxx would depend on an umaintained 9 years
> old tool when the prior versions didn't?
> 
> Also doublechecked on a Fedora 35 build: bpm-tools is not a dependency for
> building or running of mixxx.

It's not for creating and running the mixxx package, but it does involve analyzing music for beats per minute. And you check whether he analyzes beats per minute or not? And you will see that he does not.
Comment 3 sturmvogel 2022-01-30 14:48:55 CET
Created attachment 13110 [details]
Built-in BPM detection

Strange. According to https://manual.mixxx.org/1.11/en/chapters/djing_with_mixxx.html#bpm-and-beat-detection BPM detection is built-in since version 1.11.

I attached a screenshot with two test songs where the built in analysis and BPM detection worked well out of the box.
Comment 4 Nikolay Sabelnikov 2022-01-30 15:02:01 CET
Only for some reason it does not work properly, so this lib is involved.
Comment 5 Morgan Leijström 2022-01-30 15:58:41 CET
As they boast it is working better than previous per the link sturmvoger found, i think the correct action is to file a bug upstream, maybe discuss on a related forum first.

Are settings correct?
It seems you can select different analyzers
https://manual.mixxx.org/1.11/en/chapters/djing_with_mixxx.html#analyser-settings

CC: (none) => fri

Comment 6 Nikolay Sabelnikov 2022-01-30 16:12:11 CET
(In reply to Morgan Leijström from comment #5)
> As they boast it is working better than previous per the link sturmvoger
> found, i think the correct action is to file a bug upstream, maybe discuss
> on a related forum first.
> 
> Are settings correct?
> It seems you can select different analyzers
> https://manual.mixxx.org/1.11/en/chapters/djing_with_mixxx.html#analyser-
> settings

I observed on many Linux distributions that the lib inside the package did not work, so they resorted to this lib. As an alternative, you can choose. Moreover, on version 2.2.4 I see inadequate behavior of the bpm analyzer built into mixxx.
Comment 7 sturmvogel 2022-01-30 16:20:16 CET
So you should definitly file a upstream bugreport then. It's not Mageia related. And it makes absolutely no sense to add a heavily outdated and since 9 years(!!!) dead and unmaintained package to work around an upstream bug.

Also there are only two other rpm based distros which have bpm-tools. And the Opensuse package is not even official, it's only a community based 6 year old third party package.
https://pkgs.org/download/bpm-tools
Comment 8 Nikolay Sabelnikov 2022-01-30 16:28:55 CET
(In reply to sturmvogel from comment #7)
> So you should definitly file a upstream bugreport then. It's not Mageia
> related. And it makes absolutely no sense to add a heavily outdated and
> since 9 years(!!!) dead and unmaintained package to work around an upstream
> bug.
> 
> Also there are only two other rpm based distros which have bpm-tools. And
> the Opensuse package is not even official, it's only a community based 6
> year old third party package.
> https://pkgs.org/download/bpm-tools

I built with the dependency of this library, like our colleagues from ALTLinux
https://packages.altlinux.org/ru/sisyphus/srpms/mixxx/
Comment 9 Nikolay Sabelnikov 2022-01-30 20:30:02 CET
Made the assembly without this package, you were right. Thanks for the information.
Comment 10 Morgan Leijström 2022-01-30 20:33:41 CET
Meaning it works as is, and we should close the bug as invalid?
Comment 11 Nikolay Sabelnikov 2022-01-30 20:35:47 CET
(In reply to Morgan Leijström from comment #10)
> Meaning it works as is, and we should close the bug as invalid?

Yes, we can come to consensus. I can assure you that in version 2.3.1 the built-in bpm analyzer already works correctly.
Comment 12 Morgan Leijström 2022-01-30 20:37:36 CET
Great then :)

Thank you for checking and the quick and sincere communication!

Resolution: (none) => INVALID
Status: NEW => RESOLVED