Skip to content

New package: thextech-1.3.7.2#59005

Open
SkullGamer205 wants to merge 11 commits intovoid-linux:masterfrom
SkullGamer205:master
Open

New package: thextech-1.3.7.2#59005
SkullGamer205 wants to merge 11 commits intovoid-linux:masterfrom
SkullGamer205:master

Conversation

@SkullGamer205
Copy link
Copy Markdown

@SkullGamer205 SkullGamer205 commented Feb 15, 2026

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-glibc
    • x86_64-musl
    • armv6hf
    • armv7l
    • i686

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Feb 15, 2026
@SkullGamer205
Copy link
Copy Markdown
Author

SkullGamer205 commented Feb 16, 2026

About ARM (According to the author of the program)

It's need to adjust libopus's CMake build to don't use these assembly files since they are incompatible to these arm 32 builds. These arm files mostly designed for late arm32 and arm64. I had to completely disable these assemblies at some builds because of similar fails.

(it's an explicit option at AudioCodecs build that disables assembly optimisations at libopus)

(it supposed to be a rescue circle for the case when certain target unexpectedly unbuildable with these assemblies)

@SkullGamer205
Copy link
Copy Markdown
Author

SkullGamer205 commented Feb 16, 2026

Attempt to fix build on arm7l (Failed)

@SkullGamer205
Copy link
Copy Markdown
Author

Update: It can be build on x86_64-musl.

@SkullGamer205
Copy link
Copy Markdown
Author

Update: Now it can build on: armv6hf, armv7l, i686.

Comment thread srcpkgs/thextech/template Outdated
Comment thread srcpkgs/thextech/template
pkgname=thextech
version=1.3.7.2
revision=1
_subversion=1 # hotfix
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be included in the version.

Copy link
Copy Markdown
Author

@SkullGamer205 SkullGamer205 Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The versioning system will be significantly changed in the future. The second number (number "3") will be removed.
But IDK how implement hotfix into version.

Comment thread srcpkgs/thextech/template
case "$XBPS_TARGET_MACHINE" in
armv[67]*)
msg_normal "Detected ARM architecture ("$XBPS_TARGET_MACHINE")\n"
for x in ${FILESDIR}/patches/arm-disable-opus.patch; do
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We usually try to avoid conditional patches.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch is used specifically for building on architectures where building fails. However, a patch is currently being developed (which I backport) that will automatically check the architecture, and this code may no longer be necessary.

Comment thread srcpkgs/thextech/template
Comment thread srcpkgs/thextech/template Outdated
Comment thread srcpkgs/thextech/template Outdated
Comment thread srcpkgs/thextech/template Outdated
@SkullGamer205 SkullGamer205 marked this pull request as draft February 25, 2026 16:54
@SkullGamer205 SkullGamer205 marked this pull request as ready for review February 25, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package This PR adds a new package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants