Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add windows/arm{,64} support #7947

Merged
merged 1 commit into from
Sep 17, 2021
Merged

Conversation

rasa
Copy link
Member

@rasa rasa commented Sep 15, 2021

Purpose

See #7445 .

go-ole/go-ole#223 has been merged.
Once shirou/gopsutil#1132 is merged, we should be able to target windows/arm and windows/arm64.

@rasa
Copy link
Member Author

rasa commented Sep 15, 2021

Interesting that https://data.syncthing.net/ shows one Windows user already patched go-ole and gopsutil locally, and is running on arm.

@rubin55
Copy link

rubin55 commented Sep 17, 2021

Interesting that https://data.syncthing.net/ shows one Windows user already patched go-ole and gopsutil locally, and is running on arm.

:-) That would be me! I've been running Syncthing on Windows on ARM (ARM 32-bit for about 7ish months now, see #7445).

@calmh calmh merged commit 06998b3 into syncthing:main Sep 17, 2021
calmh added a commit to calmh/syncthing that referenced this pull request Sep 26, 2021
* main: (45 commits)
  build: Fix deepsource test & exclude patterns (syncthing#7969)
  build: Update all dependencies (syncthing#7968)
  lib/db: Replace SipHash with hash/maphash (syncthing#7962)
  gui: Removed superfluous quotes (fixes syncthing#7853) (syncthing#7961)
  gui, man, authors: Update docs, translations, and contributors
  lib/tlsutil: Allocate UnionedConnection in one go (syncthing#7912)
  build: Update go-ole for Windows/arm
  build: Add -arm flag to goversioninfo (syncthing#7947)
  gui, man, authors: Update docs, translations, and contributors
  gui: Sort already shared devices in edit folder modal (fixes syncthing#7940) (syncthing#7945)
  lib/model: Don't reset db while folder is running (fixes syncthing#7935) (syncthing#7936)
  lib/db: Inline sync.Once in releaser (syncthing#7938)
  all: Clarify 'Cleaning data for folder' log message (syncthing#7937)
  gui, man, authors: Update docs, translations, and contributors
  all: Unused args, retvals, assignments (syncthing#7926)
  lib/relay/protocol: Merge two Sprintf calls
  lib/discover, lib/protocol: Buffer allocation
  lib/model: Remove encryption pw on folder stop (syncthing#7925)
  lib/model: Add missing error assignment + Remove unused argument (syncthing#7922)
  gui: Move dismiss button after add/ignore buttons (syncthing#7848)
  ...
@calmh calmh added this to the v1.18.3 milestone Sep 28, 2021
@rasa rasa deleted the rasa/add_arm_support branch February 22, 2022 14:52
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Mar 10, 2023
@syncthing syncthing locked and limited conversation to collaborators Mar 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants