Skip to content

New package: libresprite-1.2#59410

Open
Kirilla39 wants to merge 1 commit intovoid-linux:masterfrom
Event-Horizon-VL:package/libresprite
Open

New package: libresprite-1.2#59410
Kirilla39 wants to merge 1 commit intovoid-linux:masterfrom
Event-Horizon-VL:package/libresprite

Conversation

@Kirilla39
Copy link
Copy Markdown

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-glibc)
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-musl (native)

@tranzystorekk tranzystorekk added the new-package This PR adds a new package label Mar 16, 2026
@hosaka
Copy link
Copy Markdown
Contributor

hosaka commented Mar 17, 2026

There should be no reason to restrict the arch to x86_64, LibreSprite should compile to other arches, at least aarch64 is supported. Trying to cross compile errors when trying to execute build/bin/gen as a part of the build process and can probably be fixed with QEMU build helper.

I have additionally hit this issue after the first install: LibreSprite/LibreSprite#598 The UI Element scaling was set to 400% by default, making it very difficult to navigate the settings menu to set it to 200%.

I would also lean towards calling this package LibreSprite, instead of librepsrite.

@classabbyamp
Copy link
Copy Markdown
Member

classabbyamp commented Mar 18, 2026

the package name is fine as-is

@Nizarjh Nizarjh force-pushed the package/libresprite branch from a3aba3e to 2474792 Compare March 18, 2026 19:46
@Nizarjh
Copy link
Copy Markdown
Contributor

Nizarjh commented Mar 18, 2026

There should be no reason to restrict the arch to x86_64, LibreSprite should compile to other arches, at least aarch64 is supported. Trying to cross compile errors when trying to execute build/bin/gen as a part of the build process and can probably be fixed with QEMU build helper.

I have additionally hit this issue after the first install: LibreSprite/LibreSprite#598 The UI Element scaling was set to 400% by default, making it very difficult to navigate the settings menu to set it to 200%.

I would also lean towards calling this package LibreSprite, instead of librepsrite.

also fix crossbuild with patch.

@Nizarjh
Copy link
Copy Markdown
Contributor

Nizarjh commented Mar 18, 2026

There should be no reason to restrict the arch to x86_64, LibreSprite should compile to other arches, at least aarch64 is supported. Trying to cross compile errors when trying to execute build/bin/gen as a part of the build process and can probably be fixed with QEMU build helper.
I have additionally hit this issue after the first install: LibreSprite/LibreSprite#598 The UI Element scaling was set to 400% by default, making it very difficult to navigate the settings menu to set it to 200%.
I would also lean towards calling this package LibreSprite, instead of librepsrite.

also fix crossbuild with patch.

tested aarch64 and aarch64-musl crossbuild. every thing work well
alsobuild for my native x86_64 and x86_64-musl. also builds well

@Kirilla39 Kirilla39 force-pushed the package/libresprite branch from 2474792 to f1394bc Compare March 18, 2026 21:09
@Kirilla39
Copy link
Copy Markdown
Author

Kirilla39 commented Mar 18, 2026

I have additionally hit this issue after the first install: LibreSprite/LibreSprite#598 The UI Element scaling was set to 400% by default, making it very difficult to navigate the settings menu to set it to 200%.

done with scaling issue with patch f1394bc

revision=1
build_style=cmake
build_helper=qemu
configure_args="-DCMAKE_EXE_LINKER_FLAGS='-lwebp'"
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.

Suggested change
configure_args="-DCMAKE_EXE_LINKER_FLAGS='-lwebp'"
configure_args="-DCMAKE_EXE_LINKER_FLAGS=-lwebp"

it's all one word anyways, shouldn't need the quotes

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.

it's all one word anyways, shouldn't need the quotes

done

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 is something that upstream should solve before we patch it

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.

ok, i removed it.

@Kirilla39 Kirilla39 force-pushed the package/libresprite branch from f1394bc to 55d5729 Compare March 22, 2026 09:49
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.

5 participants