Title: gh-90026: support XATTRs on Cygwin by philcerf · Pull Request #105075 · python/cpython · GitHub
Open Graph Title: gh-90026: support XATTRs on Cygwin by philcerf · Pull Request #105075 · python/cpython
X Title: gh-90026: support XATTRs on Cygwin by philcerf · Pull Request #105075 · python/cpython
Description: As described in #90026 (comment) and following comments, CPython’s code for XATTRs seems to already support Cygwin… mostly.
Apart from the changes made by this PR (simply compiling the relevant code), there are however also some changes needed on the Cygwin side (I guess):
Namely, they don't export XATTR_SIZE_MAX and XATTR_LIST_MAX and so a build of CPython with this commit would fail right now.
I've asked at their mailing list to have them included in their cygwin/limits.h (which I think is the proper place - Linux defines them in linux/limits.h): https://cygwin.com/pipermail/cygwin/2023-May/253750.html
Right now, he newest Version of CPython in Cygwin is 3.9.9... so unless I can get this change backported, they won't be affected by this (well and even if, ... it would be some more "motivation" to set the missing defines, I guess ;-) ).
I could of course also check whether the symbols are defined, as in:
#if defined(HAVE_SYS_XATTR_H)
# if defined(__linux__) && !defined(__FreeBSD_kernel__) && !defined(__GNU__)
# define USE_XATTRS
# include
Open Graph Description: As described in #90026 (comment) and following comments, CPython’s code for XATTRs seems to already support Cygwin… mostly. Apart from the changes made by this PR (simply compiling the relevant cod...
X Description: As described in #90026 (comment) and following comments, CPython’s code for XATTRs seems to already support Cygwin… mostly. Apart from the changes made by this PR (simply compiling the relevant cod...
Opengraph URL: https://github.com/python/cpython/pull/105075
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:55215c9f-9189-57bc-0040-68a53d1381a7 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | A6F8:3AEA09:9A709B:D362EA:6A4DDCD0 |
| html-safe-nonce | df5c50783d458759d09a379386eb3c8f375728e2e229e70f1f787ee47f9fa102 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBNkY4OjNBRUEwOTo5QTcwOUI6RDM2MkVBOjZBNEREQ0QwIiwidmlzaXRvcl9pZCI6IjgxNDYwODMwODcwNDc4NDMwMjQiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 83a20aab7c05ea8160b86bbb1f75a5406f60378a8eeef7194177b4a8c021d80e |
| hovercard-subject-tag | pull_request:1369691746 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/105075/files |
| twitter:image | https://avatars.githubusercontent.com/u/41550983?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/41550983?s=400&v=4 |
| og:image:alt | As described in #90026 (comment) and following comments, CPython’s code for XATTRs seems to already support Cygwin… mostly. Apart from the changes made by this PR (simply compiling the relevant cod... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 06b8a6144231bf3a234f1c2e9993861e07ce98a905912b114aa386c2d7e84b33 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 1d344bdb7547fe6bca17a59bb2b8aac3dc9532a0 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width