Skip to content

fix: use sed -E for macOS/BSD sed compatibility#2058

Open
Chessing234 wants to merge 1 commit into
MIT-LCP:mainfrom
Chessing234:fix/868-sed-extended-regex-portable
Open

fix: use sed -E for macOS/BSD sed compatibility#2058
Chessing234 wants to merge 1 commit into
MIT-LCP:mainfrom
Chessing234:fix/868-sed-extended-regex-portable

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

  • Concept conversion and duckdb import scripts used sed -r (GNU-only)
  • Switch to sed -E (works on GNU and BSD/macOS)

Fixes #868

Test plan

  • Run convert script on macOS without GNU sed

Made with Cursor

macOS BSD sed rejects sed -r (MIT-LCP#868).

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

sed error while generating concepts

1 participant