Skip to content

Research tools for working with patches #4704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
6 tasks done
jsjoeio opened this issue Jan 7, 2022 · 3 comments
Closed
6 tasks done

Research tools for working with patches #4704

jsjoeio opened this issue Jan 7, 2022 · 3 comments
Assignees
Milestone

Comments

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 7, 2022

Problem

We don't know what tools exist (besides quilt) for working with patches.

Solution

Research tools, build POCs with each and share findings here.

This will be closed once myself and @code-asher discuss our tool of choice.

Resources

For using quilt:

Related

Part 0/3 of #4687

TODOS

Create a POC

  • create GitHub repo jsjoeio/quilt-poc: https://github.com/jsjoeio/quilt-poc
  • add a file (use example from microsoft/vscode
  • find a file we change in our fork coder/vscode
  • set up quilt and create patch
  • record video talking through how it works
  • summarize and post here

Notes

quilt push to apply patch. quilt pop to undo patch.

@jsjoeio jsjoeio added the needs-investigation This issue needs to be further investigated label Jan 7, 2022
@jsjoeio jsjoeio self-assigned this Jan 7, 2022
@jsjoeio jsjoeio added this to the 4.0.2 milestone Jan 7, 2022
@code-asher code-asher modified the milestones: 4.0.2, 4.0.3 Jan 27, 2022
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Feb 16, 2022

Summary

quilt seems like the best tool available for managing patches. This blog shows how to perform basic tasks with quilt. The Linux man page has more information as well.

Demo

Screen.Recording.2022-02-16.at.3.22.25.PM.mov

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Feb 16, 2022

@code-asher assigning to you to watch the demo. Once you take a look, you can post questions/concerns. If we agree on using this tool, you can leave a comment and close this issue.

@jsjoeio jsjoeio assigned code-asher and unassigned jsjoeio Feb 16, 2022
@jsjoeio jsjoeio added needs-decision and removed needs-investigation This issue needs to be further investigated labels Feb 16, 2022
@code-asher
Copy link
Member

That was a fantastic demo. I say we move forward with quilt!

@jsjoeio jsjoeio assigned jsjoeio and unassigned code-asher Feb 23, 2022
@jsjoeio jsjoeio closed this as completed Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants