Skip to content

Conversation

@palfrey
Copy link

@palfrey palfrey commented Sep 7, 2025

Mass upgrade to v3 to fix #950

@Eeems
Copy link
Member

Eeems commented Sep 7, 2025

You chose v3.1 instead of v3.2 as it went to stable, which downgrades some packages. Since you are updating from v2 based images, it wouldn't matter as they would still mainly be upgraded. Let's use v3.2 instead so we can avoid the issues we swapped to stable to resolve.

@palfrey
Copy link
Author

palfrey commented Sep 7, 2025

You chose v3.1 instead of v3.2 as it went to stable, which downgrades some packages. Since you are updating from v2 based images, it wouldn't matter as they would still mainly be upgraded. Let's use v3.2 instead so we can avoid the issues we swapped to stable to resolve.

Done

Copy link
Member

@Eeems Eeems left a comment

Choose a reason for hiding this comment

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

All recipes that were changed will need their pkgver's rel (The number after the -) updated to trigger the package build.

@palfrey
Copy link
Author

palfrey commented Sep 7, 2025

This also fixes #871 as a side-effect...

@Eeems
Copy link
Member

Eeems commented Sep 7, 2025

We will need to get a new version of toltecmk pushed out to include upgrading to to resolve the build issues. It probably is worth just bumping absolutely everything when we do to make sure we didn't introduce any new issues. After we get the other PR merged I'll get the toltecmk version pushed, but it may be a bit. I'm not at home and mostly on mobile or a mobile hotspot

@Eeems
Copy link
Member

Eeems commented Sep 8, 2025

0.3.5 should be released with your changes.

@palfrey
Copy link
Author

palfrey commented Sep 8, 2025

Yay! With a small set of exceptions (folly, plato, retris) all of which have comments and links to their bug tickets, we have a working build with all the other 2.x packages on 3.x.

@Eeems
Copy link
Member

Eeems commented Sep 9, 2025

I'll take a closer look and do some testing later this week.

@palfrey
Copy link
Author

palfrey commented Sep 18, 2025

Tried deleting the Cargo.lock for those projects, and downgrading to the minimum versions, but you hit tomaka/hlua#226 (among other problems) when trying that.

@LinusCDE
Copy link
Member

Odd. I was building with the 3.2 image.
But sometimes it seemed to want to update it. No clue how cross does it. Potentially it just bumped rust in that image to latest somehow.

And yeah, seems that going back with hlua would be hard, as before there were compilation errors due to the system library changing some function definitions.

@palfrey
Copy link
Author

palfrey commented Sep 19, 2025

Odd. I was building with the 3.2 image. But sometimes it seemed to want to update it. No clue how cross does it. Potentially it just bumped rust in that image to latest somehow.

And yeah, seems that going back with hlua would be hard, as before there were compilation errors due to the system library changing some function definitions.

toltec-dev/toolchain#43 to get a newer rust with a 3.3 image appears to have sorted all of this out thankfully!

@Eeems
Copy link
Member

Eeems commented Sep 19, 2025

Alright, we have them all building now this will take a while to test as there are quite a few packages, and will require testing on both OS 2.x and 3.x for both rM1 and rM2. If anybody wants to help, feel free to comment with what you've tested. We are only really looking for a quick install/uninstall test along with launching the app to make sure it runs and appears to perform it's basic functions. There is no need for an in depth test, although if you do that, make sure to raise any issues you find on the application's repository, unless it appears to be a packaging issue.

@palfrey
Copy link
Author

palfrey commented Sep 20, 2025

I'm in progress testing things on my rm1 with OS 2.x, and will update this comment as I continue:

  • bash-completion - LGTM
  • bufshot - LGTM
  • calculator - LGTM
  • chessmarkable - LGTM
  • display - LGTM (stub installs fine)
  • doomarkable - LGTM
  • draft - LGTM
  • dumbskull - LGTM
  • evtest - LGTM
  • fbink - LGTM
  • fingerterm - LGTM
  • folly - LGTM
  • fuse - not sure how to test, installs fine
  • genie - LGTM
  • gocryptfs - runs, not really tested
  • harmony - LGTM
  • iago - runs, not really tested
  • keywriter - boots to blank screen (but I'm seeing this with the current version as well)
  • lamp - LGTM
  • libdlib - LGTM
  • libvncserver - LGTM given vnsee works
  • minesweeper - LGTM
  • mmc-utils - seems to run, not sure how tot est
  • nao - LGTM
  • netevent - LGTM
  • netsurf - LGTM
  • plato - LGTM
  • puzzles - LGTM
  • recrossable - LGTM
  • remarkable-splash - LGTM
  • remarkable-stylus - don't have the relevant hardware, installs fine
  • remux - LGTM
  • restream - LGTM
  • retris - LGTM
  • rmfakecloud-proxy - LGTM
  • rmservewacominput - boots, LGTM, not really tested
  • rpncalc - LGTM
  • sas - LGTM
  • templatectl
    Installing this gets the following and it fails to install
Configuring templatectl.
//opt/lib/opkg/info/templatectl.postinst: line 998: warning: here-document at line 21 delimited by end-of-file (wanted `UNIT')
//opt/lib/opkg/info/templatectl.postinst: line 999: syntax error: unexpected end of file

This appears to be in add-bind-mount with the UNIT bit having been indented for some reason?
Not happening on the previous version

  • vnsee - LGTM
  • wikipedia - LGTM
  • wordlet - LGTM
  • zshelf - LGTM

So with the exception of templatectl having a weirdly borked postinst, all good here.

@Eeems

This comment was marked as resolved.

@palfrey
Copy link
Author

palfrey commented Sep 20, 2025

templatectl issue is a build bug, see toltec-dev/build#60

@palfrey
Copy link
Author

palfrey commented Sep 20, 2025

templatectl install works all fine now

@Eeems
Copy link
Member

Eeems commented Oct 26, 2025

keywriter - boots to blank screen (but I'm seeing this with the current version as well)

This is because you haven't added any text yet. You would need to send keyboard events to type and add text.

fuse - not sure how to test, installs fine

You would use gocryptfs to test, as it uses the fuse module.

mmc-utils - seems to run, not sure how tot est

You can use mmc status get /dev/mmcblk1 on the rM1 and mmc status get /dev/mmcblk2 on the rM2


Testing on my rM2 on OS3:

Copy link
Member

@Eeems Eeems left a comment

Choose a reason for hiding this comment

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

See the recent comments for the changes requested. After that I'm comfortable with the level of testing so I can approve and merge this PR. Please re-request my review when you are done addressing my comments.

@palfrey
Copy link
Author

palfrey commented Oct 26, 2025

See the recent comments for the changes requested. After that I'm comfortable with the level of testing so I can approve and merge this PR. Please re-request my review when you are done addressing my comments.

Done, please re-review

@Eeems
Copy link
Member

Eeems commented Oct 26, 2025

Done, please re-review

To request a re-review, github has a feature to update the status for me to know you are actually requesting a re-review without saying something in the comments:
image
It's the arrow circle icon next to my review status.

@palfrey palfrey requested a review from Eeems October 26, 2025 23:24
@Eeems Eeems added this to the 2025-W43 Merge Window milestone Oct 26, 2025
@Eeems Eeems added packages Add or improve packages of the repository tooling Set of scripts and configuration files for building the packages labels Oct 26, 2025
@Eeems Eeems merged commit 57f496e into toltec-dev:testing Oct 27, 2025
3 checks passed
@palfrey palfrey deleted the v3-mass-upgrade branch October 27, 2025 13:57
@palfrey palfrey mentioned this pull request Oct 27, 2025
@Eeems Eeems linked an issue Oct 27, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages Add or improve packages of the repository tooling Set of scripts and configuration files for building the packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

netsurf build broken

3 participants