Bug 30338 - Macro expanded in comment on row 19: %{version}/linux-i686/xpi/ when rebuilding the firefox-l10n src.rpm
Summary: Macro expanded in comment on row 19: %{version}/linux-i686/xpi/ when rebuildi...
Status: RESOLVED WONTFIX
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 8
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-24 23:25 CEST by Kristoffer Grundström
Modified: 2022-04-26 17:25 CEST (History)
1 user (show)

See Also:
Source RPM: firefox-l10n
CVE:
Status comment:


Attachments

Description Kristoffer Grundström 2022-04-24 23:25:59 CEST
Description of problem: I see that while rebuilding the firefox-l10n-91.7.0.src.rpm you get a warning about row 19 that needs to be addressed.

Macro expanded in comment on row 19: %{version}/linux-i686/xpi/

I do know that warnings aren't fatal, but it's just this warning and nothing more.

Version-Release number of selected component (if applicable): 91.7.0

How reproducible: I have seen this only one time so far.

Steps to Reproduce:
1. Download the firefox-l10n-91.7.0 src.rpm.
2. cd to the directory where it is downloaded.
3. Install build dependencies for Firefox using dnf builddep firefox as root.
4. Run rpmbuild --rebuild firefox-l10n-91.7.0-1.mga8.src.rpm and at the beginning of the output you'll get the above mentioned warning.
Kristoffer Grundström 2022-04-24 23:26:15 CEST

CC: (none) => lovaren

Kristoffer Grundström 2022-04-24 23:26:45 CEST

Summary: Macro expanded in comment on row 19: %{version}/linux-i686/xpi/ => Macro expanded in comment on row 19: %{version}/linux-i686/xpi/ when rebuilding the firefox-l10n src.rpm

Comment 1 David Walser 2022-04-25 18:19:09 CEST
It's just a warning.  You can fix it yourself in your copy by putting another % before the %.  I can't think of any good reason you'd be rebuilding this SRPM anyway.

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

Comment 2 Kristoffer Grundström 2022-04-25 22:36:54 CEST
(In reply to David Walser from comment #1)
> It's just a warning.  You can fix it yourself in your copy by putting
> another % before the %.  I can't think of any good reason you'd be
> rebuilding this SRPM anyway.

In order to get Firefox in Swedish I need to rebuild that Firefox l10n src rpm along with the main Firefox src.rpm.

I'm rebuilding in hopes to see if the rebuilt Firefox package lowers the CPU and memory usage. I'm ruling out until I find the cause.
Comment 3 David Walser 2022-04-26 17:25:57 CEST
(In reply to Kristoffer Grundström from comment #2)
> (In reply to David Walser from comment #1)
> In order to get Firefox in Swedish I need to rebuild that Firefox l10n src
> rpm along with the main Firefox src.rpm.

No you don't.  Swedish is already included in our build.  Just install firefox-sv_SE.

Note You need to log in before you can comment on or make changes to this bug.