GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix Linux and Raspberry Pi variants displayed "Intel" in /downloads by ogoregen · Pull Request #695 · processing/processing-website

Conversation

@ogoregen

@ogoregen ogoregen commented Jun 24, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Was caused by getBit() returning Intel for all package names containing x64. Also changed the label of the Windows variant from Intel to 64-bit as the former was misleading/incorrect.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@ogoregen
Fix getBit returning Intel for Linux and Raspberry Pi packages
f667685
@catilac
catilac self-requested a review June 24, 2026 16:37
@netlify

netlify Bot commented Jun 24, 2026

Copy link
Copy Markdown

✅ Deploy Preview for java-processing-faf822 ready!

Name Link
🔨 Latest commit f667685
🔍 Latest deploy log https://app.netlify.com/projects/java-processing-faf822/deploys/6a3c0432e92d39000776747b
😎 Deploy Preview https://deploy-preview-695--java-processing-faf822.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@SableRaf

SableRaf commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Flagging for review: should check how the new links get picked up by Fathom analytics.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@catilac

catilac commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Thanks for calling that out! I was about to tag you @SableRaf in regards to the visual elements. But sounds like it's good. I'll check the analytics

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

catilac
catilac reviewed Jun 25, 2026
View reviewed changes

@catilac catilac left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

inside of src/templates/download.js there is tracking logic. We need to make sure we're correctly tracking. I will make a PR for this @ogoregen since I need to create some tracking codes.

After that gets merged into this branch, we can merge into main if all looks well.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@SableRaf

SableRaf commented Jun 25, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Another thing is the info tooltip for each distribution could include an explanation so ppl can tell the difference between each build.

I was about to tag you @SableRaf in regards to the visual elements.

I've been thinking about this too. I'd suggest putting the Snap button at the top since that's the recommended install for beginners, and the other two buttons could use a secondary color. People who want the .deb or flatpak versions likely know what they need, but a beginner downloading the wrong package is likely to get stuck.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ogoregen

ogoregen commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Got it, @catilac.

I think Snap isn't the one we should highlight as it serves only Ubuntu users. Flatpak is the option that is both easy and that serves the most number of distros @SableRaf.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@SableRaf

SableRaf commented Jun 25, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Does Flatpak take care of MIME file association, setting up the JAVA environment variable that the VS Code extension uses to locate Processing, and configure support for the pde:// URI schema?

I know that some distributions (like Mint) choose to disable snapd by default, but you can still re-enable it.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@catilac

catilac commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

@ogoregen @SableRaf -- would it make sense to move this discussion into another issue? it sounds less urgent than fixing these links. let's move this discussion into discord and we can distill it into a task we can assign to @ogoregen?

SableRaf and ogoregen reacted with thumbs up emoji
All reactions
  • 👍 2 reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@duskvirkus

duskvirkus commented Aug 25, 2026

Copy link
Copy Markdown
Screenshot_20260825_120544

Came here to file an issue about this. Tool tip would be great but right now this is very confusing, I had to go into devtools to figure out which one I wanted to download.

I think it would be great to merge this fix. @SableRaf think you would have the ability to do that.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@SableRaf

SableRaf commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

@duskvirkus This should be addressed by the new download flow in #683, however it is currently blocked by processing/processing4#1425

@catilac If the download flow is likely to take longer, it may be worth patching the buttons separately in the meantime. What do you think?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@catilac

catilac commented Aug 27, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

@SableRaf Yeah I think patching makes sense for now, I'm currently blocked by other tasks on the way to ars

SableRaf reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ulgens

ulgens commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

This should be addressed by the new download flow in #683, however it is currently blocked by

I was about to comment on processing/processing4#1425, saying it would be better to fix the links on the website first but now checking the discussion here, it seems we have a circular dependency. Any chance updating the links without #683 and processing/processing4#1425 ?

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ulgens ulgens mentioned this pull request Aug 28, 2026
Add an install.sh for easy install processing/processing4#1425
Open
@ulgens

ulgens commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

I just checked the deployment preview and everything looks fine. I think this can be merged without any blockers.

ogoregen reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

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

Reviewers

@catilac catilac catilac left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@ogoregen @SableRaf @catilac @duskvirkus @ulgens
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.