Skip to content

[mpg123] Upgrade to 1.26.3-1#16910

Merged
ras0219-msft merged 2 commits intomicrosoft:masterfrom
evpobr:mpg123-1.26.3-1
Apr 2, 2021
Merged

[mpg123] Upgrade to 1.26.3-1#16910
ras0219-msft merged 2 commits intomicrosoft:masterfrom
evpobr:mpg123-1.26.3-1

Conversation

@evpobr
Copy link
Contributor

@evpobr evpobr commented Mar 27, 2021

Fix invalid MPG123_API_VERSION value in mpg123.h.in for Windows platform. It was equal to @API_VERSION@, now it is read from configure.ac and set to correct value.

See also libsndfile/libsndfile#499.

@PhoebeHui PhoebeHui self-assigned this Mar 27, 2021
@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Mar 27, 2021
@evpobr
Copy link
Contributor Author

evpobr commented Mar 27, 2021

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 16910 in repo microsoft/vcpkg

Fix invalid MPG123_API_VERSION value in mpg123.h.in for Windows
platform. It was equal to @API_VERSION@, now it is read from
configure.ac and set to correct value.

See also libsndfile/libsndfile#499.
@evpobr
Copy link
Contributor Author

evpobr commented Mar 31, 2021

@PhoebeHui , can you restart checks?

evpobr added a commit to libsndfile/libsndfile that referenced this pull request Mar 31, 2021
@PhoebeHui
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@evpobr
Copy link
Contributor Author

evpobr commented Mar 31, 2021

@PhoebeHui , is this my bug?

@PhoebeHui
Copy link
Contributor

@evpobr, this is not a bug, since you have a new commit, the baseline hash changed, could you update the baseline version by executing command './vcpkg x-add-version --overwrite-version mpg123', and then submit the changes again?

@evpobr
Copy link
Contributor Author

evpobr commented Apr 1, 2021

@PhoebeHui , ready.

@PhoebeHui PhoebeHui added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Apr 1, 2021
@ras0219-msft ras0219-msft merged commit 4ec290c into microsoft:master Apr 2, 2021
@ras0219-msft
Copy link
Contributor

LGTM, thanks for the fix, looking forward to the libsndfile improvements :)

@evpobr
Copy link
Contributor Author

evpobr commented Apr 3, 2021

@PhoebeHui , @ras0219-msft , current stable version is 1.26.5. Should i add 1.26.4 too?

@evpobr evpobr deleted the mpg123-1.26.3-1 branch April 3, 2021 04:58
@PhoebeHui
Copy link
Contributor

@evpobr, we prefer to update it with lastest stable revision, thanks again for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants