Skip to content
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

Investigate changes to archive.rs #1957

Closed
tedinski opened this issue Dec 3, 2022 · 0 comments · Fixed by #2459
Closed

Investigate changes to archive.rs #1957

tedinski opened this issue Dec 3, 2022 · 0 comments · Fixed by #2459
Labels
[C] Internal Tracks some internal work. I.e.: Users should not be affected.

Comments

@tedinski
Copy link
Contributor

tedinski commented Dec 3, 2022

When we added rlib support we copied some code from archive.rs.

Looks like a recently merged PR in rust upstream impacted this stuff a lot:

Need to see if this will help simplify our code a lot. Plus help whoever gets the rust toolchain update task this week, since it the above PR might impact that too.

@tedinski tedinski added the [C] Internal Tracks some internal work. I.e.: Users should not be affected. label Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Internal Tracks some internal work. I.e.: Users should not be affected.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant