Skip to content

stepmania: fix build with gcc 12, ship ffmpeg 2.1.3 with distfiles#40783

Merged
paper42 merged 1 commit into
masterfrom
unknown repository
Dec 3, 2022
Merged

stepmania: fix build with gcc 12, ship ffmpeg 2.1.3 with distfiles#40783
paper42 merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Nov 26, 2022

Copy link
Copy Markdown

If ffmpeg with specific version is not present in the work source directory, cmake attempts to clone its repository with git and check out version 2.1.3. This approach presents two issues:

  1. It can no longer be cloned using git:// for github (see FIx "protocol on port 9418 is no longer supported" stepmania/stepmania#2200) which results in connection timeout and build failure as the result.

  2. Downloaded tarball is erased after successful build since it is considered to be part of the source code.
    Its retrieval and storage for future use should be done by xbps-src.

cc: @the-maldridge as the current maintainer of the package.

Testing the changes

  • I tested the changes in this PR: briefly

Local build testing

  • I built this PR locally for my native architecture, x86_64

Comment thread srcpkgs/stepmania/patches/newgcc.patch Outdated
Comment thread srcpkgs/stepmania/patches/flags.patch Outdated
Comment thread srcpkgs/stepmania/template Outdated
Comment thread srcpkgs/stepmania/template Outdated
Comment thread srcpkgs/stepmania/template Outdated
Comment thread srcpkgs/stepmania/template Outdated
If ffmpeg with specific version is not present in the work source
directory, cmake attempts to clone its repository with git and
check out version 2.1.3. This approach presents two issues:

1. It can no longer be cloned using git:// for github (see
stepmania/stepmania#2200) which results in
connection timeout and build failure as the result.

2. Downloaded tarball is erased after successful build since it is
considered to be part of the source code.
Its retrieval and storage for future use should be done by xbps-src.
@paper42 paper42 merged commit 8078afb into void-linux:master Dec 3, 2022
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

Successfully merging this pull request may close these issues.

1 participant