Skip to content

tidy-up: miscellaneous#1842

Closed
vszakats wants to merge 5 commits intolibssh2:masterfrom
vszakats:tidy2604
Closed

tidy-up: miscellaneous#1842
vszakats wants to merge 5 commits intolibssh2:masterfrom
vszakats:tidy2604

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Apr 9, 2026

  • configure: fix indent, whitespace, drop empty comments.
  • .gitignore: drop .DS_Store (again)
    Keep this list for files created by libssh2. For other files, the
    per-machine global Git ignore file is a better place, and it fixes
    this for all repos, not just libssh2.

https://github.com/libssh2/libssh2/pull/1842/files?w=1

vszakats added 5 commits April 9, 2026 16:06
libssh2 does not create this and it's not libssh2-specific. The best
place for this is the global .gitignore on the local macOS machine.

Follow-up to 2b71623 libssh2#1803
Follow-up to 4f9e6e0 libssh2#1718
@willco007
Copy link
Copy Markdown
Member

Please don't remove.ds_store from git ignore.

@vszakats
Copy link
Copy Markdown
Member Author

vszakats commented Apr 9, 2026

Please don't remove.ds_store from git ignore.

Can you add it to your local ignore file? Why add to libssh2?

@willco007
Copy link
Copy Markdown
Member

It's fine, I'll do that. I'd prefer if libssh2 had 'logical defaults' out of the box however.

@vszakats
Copy link
Copy Markdown
Member Author

vszakats commented Apr 9, 2026

That may be a PR for Git itself, fixing it for all repos. (Not even dreaming Apple fixing this in addition.)

Thanks for fixing locally.

@vszakats vszakats closed this in 3d4ff6b Apr 10, 2026
@vszakats vszakats deleted the tidy2604 branch April 10, 2026 16:39
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.

2 participants