Skip to content

Wifi fallback fix#7

Open
privatereese wants to merge 2 commits into
experimentalfrom
wifi-fallback-fix
Open

Wifi fallback fix#7
privatereese wants to merge 2 commits into
experimentalfrom
wifi-fallback-fix

Conversation

@privatereese

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the site build inputs to pick up a WiFi fallback fix from the tecff Gluon packages feed and improves the build Makefile’s portability across non-Linux environments.

Changes:

  • Pin tecff/gluon-packages to the wifi-fallback-fix branch and a new commit.
  • Make the default JOBS autodetection work on BSD/macOS and other POSIX systems (fallback to sysctl/getconf).
  • Replace GNU ln -T usage with a more portable “remove then symlink” approach for ${GLUON_BUILD_DIR}/site.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
modules Switches the tecff feed branch/commit to the WiFi fallback fix revision.
Makefile Improves cross-platform CPU detection and makes the site symlink step portable without GNU ln -T.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rotanid rotanid left a comment

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.

do we know the build change works with Debian 11/12/13 ?
if yes, it's fine with me.

Comment thread modules
Comment on lines -4 to -5
PACKAGES_TECFF_BRANCH=main
PACKAGES_TECFF_COMMIT=8b81ae7bda36e99e1fbe5001ae9559bc41bc215f

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.

nope. first has to be merged, then modules will be pushed in main branch.

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