Title: Dual declaration of directories in Makefile is a foot gun · Issue #102031 · python/cpython · GitHub
Open Graph Title: Dual declaration of directories in Makefile is a foot gun · Issue #102031 · python/cpython
X Title: Dual declaration of directories in Makefile is a foot gun · Issue #102031 · python/cpython
Description: I'd estimate that 90% of the times that I introduce a new directory to the Python project, I forget that Python requires a two-phase operation to add new directories: Add the directory and files to the Git repo. Add a reference to the ne...
Open Graph Description: I'd estimate that 90% of the times that I introduce a new directory to the Python project, I forget that Python requires a two-phase operation to add new directories: Add the directory and files to...
X Description: I'd estimate that 90% of the times that I introduce a new directory to the Python project, I forget that Python requires a two-phase operation to add new directories: Add the directory and file...
Opengraph URL: https://github.com/python/cpython/issues/102031
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Dual declaration of directories in Makefile is a foot gun","articleBody":"I'd estimate that 90% of the times that I introduce a new directory to the Python project, I forget that Python requires a two-phase operation to add new directories:\r\n\r\n- Add the directory and files to the Git repo.\r\n- Add a reference to the new directory to the Makefile.\r\n\r\nIf one completes only the first step, the tests will pass locally and in CI, but will fail in buildbots after merging. See https://github.com/python/cpython/pull/102010 for the most recent example.\r\n\r\nIdeally, the build system should be reconfigured such that only one phase is required to add a new directory to the build.\r\n\r\nShort of that, the CI builds should somehow check that the introduction of a new directory either triggers the buildbot \"installed\" builds or otherwise checks that the new directories are included in any source bundle.\r\n\r\nIt should not be a common occurrence to introduce a directory, pass CI, and then fail buildbots.","author":{"url":"https://github.com/jaraco","@type":"Person","name":"jaraco"},"datePublished":"2023-02-18T22:48:43.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/102031/cpython/issues/102031"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:954cdea1-74fc-1239-9414-e9b9e601062b |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 893E:3569F3:1AF6BB:25EEEF:696A05F6 |
| html-safe-nonce | 91b01a95d63a8514c726c3030341a5611c107726282ed6e5c236799104539487 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4OTNFOjM1NjlGMzoxQUY2QkI6MjVFRUVGOjY5NkEwNUY2IiwidmlzaXRvcl9pZCI6IjcwNDI5MzUzMDEyODk0MTIwODciLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 7bc36c35442f0726f3677a3e088084b244a21f45608b6163e28e8f02512cc1bb |
| hovercard-subject-tag | issue:1590493086 |
| github-keyboard-shortcuts | repository,issues,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/_view_fragments/issues/show/python/cpython/102031/issue_layout |
| twitter:image | https://opengraph.githubassets.com/65332d3a8aa633866a7a5bc98140ceeec076a6c45c0ce9bbe4d354670192ee2d/python/cpython/issues/102031 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/65332d3a8aa633866a7a5bc98140ceeec076a6c45c0ce9bbe4d354670192ee2d/python/cpython/issues/102031 |
| og:image:alt | I'd estimate that 90% of the times that I introduce a new directory to the Python project, I forget that Python requires a two-phase operation to add new directories: Add the directory and files to... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jaraco |
| hostname | github.com |
| expected-hostname | github.com |
| None | 578c119ff0247c8b2f2491fbf4fc0395cdf909d4df66598cebdc96ddfc4418dc |
| turbo-cache-control | no-preview |
| 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 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 671c2f67171dbced24284331f3133a613d08c366 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width