831044 – media-libs/xvid-1.3.7: replaces -mcpu with -mtune, leading to build failure.

Bug 831044 - media-libs/xvid-1.3.7: replaces -mcpu with -mtune, leading to build failure.

Summary: media-libs/xvid-1.3.7: replaces -mcpu with -mtune, leading to build failure.

Status:	CONFIRMED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Gentoo Media-video project

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2022-01-12 02:41 UTC by Georgy Yakovlev
Modified:	2024-04-04 23:17 UTC (History)
CC List:	2 users (show)

See Also:	831211
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

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

Description Georgy Yakovlev archtester

2022-01-12 02:41:05 UTC

so build/generic/configure:5025:

has this code (in couple of places)

CFLAGS=`echo $CFLAGS | sed s,"-mcpu","-mtune",g`


and it replaces valid -mcpu with -mtune which does not support all same options

on arm64 it's perfectly legal to specify CPU crypto extention support like this


-mcpu=cortex-a72.cortex-a53+crypto

but xvid replaces -mcpu with -tune where it's not legal

cc1: error: unknown value 'cortex-a72.cortex-a53+crypto' for '-mtune' ; did you mean 'cortex-a72.cortex-a53'?

so basically it bails out if it sees the +crypto part.


source is here
http://websvn.xvid.org/cvs/viewvc.cgi/tags/release-1_3_7/xvidcore/build/generic/configure.in?view=markup

maybe let's just rip it out?

Comment 1 David Racine 2024-04-04 23:17:54 UTC

I have a fix in my personal overlay:
https://github.com/bassdr/gentoo-local-overlay/tree/master/media-libs/xvid

I remove the mcpu -> mtunes replacement completely, and it just works.