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

perl.prov mistakenly adds embedded Parse::Yapp::Driver #1

Open
jirislaby opened this issue Oct 6, 2022 · 3 comments
Open

perl.prov mistakenly adds embedded Parse::Yapp::Driver #1

jirislaby opened this issue Oct 6, 2022 · 3 comments

Comments

@jirislaby
Copy link

jirislaby commented Oct 6, 2022

Math-Calc-Units uses Parse::Yapp::Driver and contains its generated grammar:
https://metacpan.org/release/SFINK/Math-Calc-Units-1.07/source/Units/Grammar.pm

There is:

package Math::Calc::Units::Grammar;
use vars qw ( @ISA );
use strict;
 
@ISA= qw ( Parse::Yapp::Driver );
#Included Parse/Yapp/Driver.pm file----------------------------------------
{
#
# Module Parse::Yapp::Driver
#
# This module is part of the Parse::Yapp package available on your
# nearest CPAN
...
 
package Parse::Yapp::Driver;
...
$VERSION = '1.04';

perl.prov generates:

perl(Parse::Yapp::Driver) = 1.04

It should likely (somehow) ignore embedded packages. Or at least provide a blacklist feature (to supply Parse::Yapp::Driver in this very case).

@pmatilai
Copy link
Member

Patches welcome.

Until then, use the filtering mechanism(s) available: https://rpm-software-management.github.io/rpm/manual/dependency_generators.html

@ffesti
Copy link
Contributor

ffesti commented Dec 6, 2023

Those generators are something we would happily hand over to a Perl community maintained perl-rpm-macros package.

See also #2747

@ffesti ffesti removed their assignment Dec 6, 2023
@pmatilai pmatilai transferred this issue from rpm-software-management/rpm Mar 8, 2024
@stoecker
Copy link
Collaborator

For openSUSE we use patches which change source so that it's no longer detected (usually with a line break or similar). That's ugly and thus never reported upstream. Maybe we can add a keyword for these cases (like # no-rpm-provides) which upstream authors would accept?

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

No branches or pull requests

4 participants