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

dockerfiles: change from xalan to xsltproc #254

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

jbech-linaro
Copy link
Contributor

Given the varying xalan versions and their distinct behavior on different distributions, adopting xsltproc as a replacement offers consistent multi-distro support, as it appears to be compatible with most platforms.

I haven't tested the 20.04 Dockerfile, if @jforissier has time and ability to do so, it'd be appreciated.

Related to: OP-TEE/optee_test#761

@jenswi-linaro
Copy link
Contributor

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

@jforissier
Copy link
Contributor

jforissier commented Nov 14, 2024

I haven't tested the 20.04 Dockerfile, if @jforissier has time and ability to do so, it'd be appreciated.

Yes, it does work fine.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

Given the varying xalan versions and their distinct behavior on
different distributions, adopting xsltproc as a replacement offers
consistent multi-distro support, as it appears to be compatible with
most platforms.

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
@jbech-linaro
Copy link
Contributor Author

@jenswi-linaro @jforissier , thanks for acking and thanks for testing on 20.04, much appreciated.

@jbech-linaro jbech-linaro merged commit b9ff227 into OP-TEE:master Nov 14, 2024
1 check passed
@jbech-linaro jbech-linaro deleted the gp-package-fix branch November 14, 2024 14:58
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.

3 participants