Skip to content

Add DelimitedFiles package and compat for it to packages#66039

Merged
KristofferC merged 2 commits into
masterfrom
kc/delimited_files_compat
Aug 11, 2022
Merged

Add DelimitedFiles package and compat for it to packages#66039
KristofferC merged 2 commits into
masterfrom
kc/delimited_files_compat

Conversation

@KristofferC

Copy link
Copy Markdown
Member

The plan is to move out DelimitedFiles from being an stdlib to being an external package. However, there is no compat on standard libraries. When people start using DelimitedFiles as a normal package, the lack of compat will be problematic.

As described in JuliaLang/julia#45540 (comment), this mirrors the compat for Julia to that of DelimitedFiles which should make sense for existing versions, new versions of packages pre 1.9 should have compat on DelimitedFiles that mirrors their Julia compat, packages only supporting 1.9+ are free to have any compat on DelimitedFiles.

KristofferC and others added 2 commits August 10, 2022 22:09
UUID: 8bb1440f-4735-579b-a4ab-409b98df4dab
Repo: https://github.com/JuliaData/DelimitedFiles.jl.git
Tree: 19b1417ff479c07e523fcbf2fd735a3fde3d1ab3

Registrator tree SHA: 8e1a5ac2695627143951512d700c7e3c445102ec

(cherry picked from commit 9539634)
@KristofferC KristofferC temporarily deployed to stopwatch August 10, 2022 20:21 Inactive
@KristofferC

Copy link
Copy Markdown
Member Author

The PR here should be safe in the sense that the info here should be completely ignored on pre 1.9 Julias and it is also easy to revert since you would just remove all the lines with DelimitedFiles in Compat files.

@DilumAluthge DilumAluthge left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

To be safe, it would be nice to do a PkgEval run using this branch of the registry, just to make sure that we aren't missing anything.

@KristofferC

Copy link
Copy Markdown
Member Author

To be safe, it would be nice to do a PkgEval run using this branch of the registry, just to make sure that we aren't missing anything.

JuliaLang/julia#45540 (comment) ran on a branch that included this PR and didn't show any problems.

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.

3 participants