Skip to content

Comments

Add infrastructure to sync 16 upstream issues from paulnasca/paulstretch_cpp#1

Merged
peternicholls merged 5 commits intomasterfrom
copilot/create-issues-from-upstream
Dec 8, 2025
Merged

Add infrastructure to sync 16 upstream issues from paulnasca/paulstretch_cpp#1
peternicholls merged 5 commits intomasterfrom
copilot/create-issues-from-upstream

Conversation

Copy link

Copilot AI commented Nov 8, 2025

Provides scripts and documentation to create 16 issues from upstream paulnasca/paulstretch_cpp repository with proper attribution and formatting.

Files Added

  • SYNC_UPSTREAM_ISSUES.md - Full issue content for all 16 upstream issues
  • scripts/sync_upstream_issues.sh - Bash script using gh CLI (preferred)
  • scripts/create_upstream_issues.py - Python script using PyGithub API
  • scripts/README.md - Usage instructions and troubleshooting
  • CREATE_ISSUES_INSTRUCTIONS.md - Quick-start guide
  • .gitignore - Excludes Python cache

Issue Format

Each issue includes:

  • Original upstream link
  • Author attribution (e.g., @dblanchemain)
  • Complete original body
  • [Synced from upstream] prefix

Execution

# Using GitHub CLI
./scripts/sync_upstream_issues.sh

# Using Python API
export GITHUB_TOKEN="..."
python3 scripts/create_upstream_issues.py

Scripts use heredoc/file-based approach to safely handle special characters in issue bodies. Both syntax-validated.

Issues Covered

16 total: segfault (paulnasca#18), GCC14 undefined behavior (PR#17), Mac signing (paulnasca#15), mxml 3.0 compat (PR#12), CMake build (PR#8), FLAC support (PR#7), and 10 others spanning documentation, build fixes, and feature requests.

Original prompt

Create 16 issues synced from upstream paulnasca/paulstretch_cpp repository. For each issue, create it with the original title and add the original description/body with a link to the upstream issue and original author information.

Issues to create:

  1. Title: "Paulstretch segmentation fault" - Link: Paulstretch segmentation fault paulnasca/paulstretch_cpp#18 - Author: dblanchemain
  2. Title: "No return in a function expecting one... Triggers undefined behavior with GCC14" - Link: No return in a function expecting one... Triggers undefined behavior with GCC14 paulnasca/paulstretch_cpp#17 - Author: ttsiodras
  3. Title: "Clear path to download" - Link: Clear path to download paulnasca/paulstretch_cpp#16 - Author: hebronwatson
  4. Title: "Mac Builds are unsigned" - Link: Mac Builds are unsigned paulnasca/paulstretch_cpp#15 - Author: amias-channer
  5. Title: "Multiple audio file support?" - Link: Multiple audio file support? paulnasca/paulstretch_cpp#14 - Author: Jamaldude
  6. Title: "fix compatibility with mxml 3.0" - Link: fix compatibility with mxml 3.0 paulnasca/paulstretch_cpp#12 - Author: afontenot
  7. Title: "Feature request: Use frequency shift to generate binaural beats" - Link: Feature request: Use frequency shift to generate binaural beats paulnasca/paulstretch_cpp#10 - Author: nebosite
  8. Title: "PaulStretch to Homebrew Cask?" - Link: PaulStretch to Homebrew Cask? paulnasca/paulstretch_cpp#9 - Author: esaruoho
  9. Title: "C make build system" - Link: C make build system paulnasca/paulstretch_cpp#8 - Author: Chaircrusher
  10. Title: "ENH: Add flac reading and writing" - Link: ENH: Add flac reading and writing paulnasca/paulstretch_cpp#7 - Author: Chaircrusher
  11. Title: "Integration improvements" - Link: Integration improvements paulnasca/paulstretch_cpp#6 - Author: trebmuh
  12. Title: "Update Control.cpp (typo)" - Link: Update Control.cpp (typo) paulnasca/paulstretch_cpp#5 - Author: trebmuh
  13. Title: "Mac OS X version" - Link: Mac OS X version paulnasca/paulstretch_cpp#4 - Author: padsbanger
  14. Title: "ArchLinux build error" - Link: ArchLinux build error paulnasca/paulstretch_cpp#3 - Author: smoge
  15. Title: "missing zlib linkage" - Link: missing zlib linkage paulnasca/paulstretch_cpp#2 - Author: iioflow
  16. Title: "First few seconds at any stretch have intense distortion" - Link: First few seconds at any stretch have intense distortion paulnasca/paulstretch_cpp#1 - Author: dancrew32

For each issue, create it as a GitHub issue in peternicholls/paulstretch_cpp with the format:
[Synced from upstream]

Original Issue: [link to upstream]
Original Author: @[username]

Include any relevant details or body content in the issue description.

This pull request was created as a result of the following prompt from Copilot chat.

Create 16 issues synced from upstream paulnasca/paulstretch_cpp repository. For each issue, create it with the original title and add the original description/body with a link to the upstream issue and original author information.

Issues to create:

  1. Title: "Paulstretch segmentation fault" - Link: Paulstretch segmentation fault paulnasca/paulstretch_cpp#18 - Author: dblanchemain
  2. Title: "No return in a function expecting one... Triggers undefined behavior with GCC14" - Link: No return in a function expecting one... Triggers undefined behavior with GCC14 paulnasca/paulstretch_cpp#17 - Author: ttsiodras
  3. Title: "Clear path to download" - Link: Clear path to download paulnasca/paulstretch_cpp#16 - Author: hebronwatson
  4. Title: "Mac Builds are unsigned" - Link: Mac Builds are unsigned paulnasca/paulstretch_cpp#15 - Author: amias-channer
  5. Title: "Multiple audio file support?" - Link: Multiple audio file support? paulnasca/paulstretch_cpp#14 - Author: Jamaldude
  6. Title: "fix compatibility with mxml 3.0" - Link: fix compatibility with mxml 3.0 paulnasca/paulstretch_cpp#12 - Author: afontenot
  7. Title: "Feature request: Use frequency shift to generate binaural beats" - Link: Feature request: Use frequency shift to generate binaural beats paulnasca/paulstretch_cpp#10 - Author: nebosite
  8. Title: "PaulStretch to Homebrew Cask?" - Link: PaulStretch to Homebrew Cask? paulnasca/paulstretch_cpp#9 - Author: esaruoho
  9. Title: "C make build system" - Link: C make build system paulnasca/paulstretch_cpp#8 - Author: Chaircrusher
  10. Title: "ENH: Add flac reading and writing" - Link: ENH: Add flac reading and writing paulnasca/paulstretch_cpp#7 - Author: Chaircrusher
  11. Title: "Integration improvements" - Link: Integration improvements paulnasca/paulstretch_cpp#6 - Author: trebmuh
  12. Title: "Update Control.cpp (typo)" - Link: Update Control.cpp (typo) paulnasca/paulstretch_cpp#5 - Author: trebmuh
  13. Title: "Mac OS X version" - Link: Mac OS X version paulnasca/paulstretch_cpp#4 - Author: padsbanger
  14. Title: "ArchLinux build error" - Link: ArchLinux build error paulnasca/paulstretch_cpp#3 - Author: smoge
  15. Title: "missing zlib linkage" - Link: missing zlib linkage paulnasca/paulstretch_cpp#2 - Author: iioflow
  16. Title: "First few seconds at any stretch have intense distortion" - Link: First few seconds at any stretch have intense distortion paulnasca/paulstretch_cpp#1 - Author: dancrew32

For each issue, create it as a GitHub issue in peternicholls/paulstretch_cpp with the format:
[Synced from upstream]

Original Issue: [link to upstream]
Original Author: @[username]

Include any relevant details or body content in the issue description.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 4 commits November 8, 2025 07:22
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Co-authored-by: peternicholls <519525+peternicholls@users.noreply.github.com>
Copilot AI changed the title [WIP] Create 16 issues synchronized from upstream repository Add infrastructure to sync 16 upstream issues from paulnasca/paulstretch_cpp Nov 8, 2025
Copilot AI requested a review from peternicholls November 8, 2025 07:29
@peternicholls peternicholls marked this pull request as ready for review December 8, 2025 11:00
Copilot AI review requested due to automatic review settings December 8, 2025 11:00
@peternicholls peternicholls merged commit 5dbed80 into master Dec 8, 2025
2 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR provides infrastructure to sync 16 issues from the upstream paulnasca/paulstretch_cpp repository into the peternicholls/paulstretch_cpp fork. The implementation includes automation scripts, comprehensive documentation, and complete issue content preservation with proper attribution.

Key Changes

  • Adds two automated scripts (Bash using GitHub CLI and Python using PyGithub API) to create 16 upstream issues
  • Provides complete documentation in markdown files with usage instructions, troubleshooting guides, and full issue content
  • Updates .gitignore to exclude Python cache and IDE files

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
scripts/sync_upstream_issues.sh Bash script using heredocs to create 16 GitHub issues via gh CLI with proper escaping of special characters
scripts/create_upstream_issues.py Python script using PyGithub library as alternative implementation for issue creation
scripts/README.md Comprehensive usage guide covering both scripts, prerequisites, troubleshooting, and token setup
SYNC_UPSTREAM_ISSUES.md Complete formatted content of all 16 upstream issues for manual reference or creation
CREATE_ISSUES_INSTRUCTIONS.md Quick-start guide with three execution options (GitHub CLI, Python API, or manual)
.gitignore Standard Python, IDE, and OS-specific exclusions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

**Original Issue:** https://github.com/paulnasca/paulstretch_cpp/pull/8
**Original Author:** @Chaircrusher

This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against.
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".

Suggested change
This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against.
This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against.

Copilot uses AI. Check for mistakes.
"title": "C make build system",
"upstream_link": "https://github.com/paulnasca/paulstretch_cpp/pull/8",
"author": "Chaircrusher",
"body": "This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against."
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".

Suggested change
"body": "This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against."
"body": "This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against."

Copilot uses AI. Check for mistakes.
**Original Issue:** https://github.com/paulnasca/paulstretch_cpp/pull/8
**Original Author:** @Chaircrusher

This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against.
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammatical error in the issue body. The phrase "This should all (still) building" is missing the word "be" and should read "This should all (still) be building on Mac OS X".

Suggested change
This simply adds the CMake build system. This should all (still) building on Mac OS X, but I no longer have a system to test this build environment against.
This simply adds the CMake build system. This should all (still) be building on Mac OS X, but I no longer have a system to test this build environment against.

Copilot uses AI. Check for mistakes.

## Notes

- These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates.
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The statement "These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues" is contradictory and confusing. Idempotent operations can be run multiple times without creating duplicates. The correct phrasing should be: "These scripts are NOT idempotent, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates."

Suggested change
- These scripts are idempotent-safe, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates.
- These scripts are NOT idempotent, meaning running them multiple times will create duplicate issues. Only run once unless you want duplicates.

Copilot uses AI. Check for mistakes.
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