3 ms·
Speaking of poor programming practices, how about copy-pasting code without understanding it? Notice the author's mention of how many packages with -Ofast trace
by cratermoon 2y ago
Speaking of poor programming practices, how about copy-pasting code without understanding it?
Notice the author's mention of how many packages with -Ofast trace their lineage back through this comment:
# Initially copied from
# https://github.com/actions/starter-workflows/blob/main/ci/python-package.yml
# And later based on the version jamadden updated at
# gevent/gevent, and then at zodb/relstorage and zodb/perfmetrics
Although the linked config was modified 2 years ago to removed the unsafe math options[1],
the copy-paste propagated before then.
Naturally anyone asking CoPilot, ChatGPT, or other modern LLM-based interface for config code will likely get something based on this, with the problematic -Ofast option included.
[1] https://github.com/zopefoundation/meta/commit/9c07520e90d9a12e82c1b26501e20ba86e21312c https://github.com/zopefoundation/meta/commit/9c07520e90d9a1...