Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

DilumAluthge
Copy link
Member

No description provided.

@DilumAluthge DilumAluthge added the DO NOT MERGE Do not merge this PR! label Sep 12, 2025
@DilumAluthge
Copy link
Member Author

@nanosoldier runtests(["LinearAlgebra", "Pkg"])

@nanosoldier
Copy link
Collaborator

The package evaluation job you requested has completed - no new issues were detected.
The full report is available.

Report summary

~ Packages that at least loaded

2 packages successfully loaded on the previous version too.

@DilumAluthge
Copy link
Member Author

Hmmm, this doesn't look correct.

Pkg is a standard library in this Julia build.


################################################################################
# Loading
#

Loading Pkg...

Loading completed after 2.08s

PkgEval succeeded after 9.83s

It looks like it only tests that it can load Pkg. But we want to actually run the Pkg test suite.

@DilumAluthge
Copy link
Member Author

The logs show the line # Loading.

Based on https://github.com/JuliaCI/PkgEval.jl/blob/master/scripts/evaluate.jl, I think we are hitting https://github.com/JuliaCI/PkgEval.jl/blob/524847dc78c801a51a16ae7e747ebe19f8fc30a4/scripts/evaluate.jl#L184-L185

But this means that config.goal === :load is true.

So, why is config.goal getting set to :load???

@DilumAluthge
Copy link
Member Author

I suppose I should write a sentence or two on why I want this to work.

Now that we have merged JuliaCI/julia-buildkite#483, external stdlibs will not be tested on PRs to Julia master. So I'd like some way to manually trigger external stdlib tests on a PR. And PkgEval seems like the easiest mechanism to use.

@DilumAluthge
Copy link
Member Author

@nanosoldier runtests(["LinearAlgebra", "Pkg"], configuration = (goal=:test,), vs_configuration = (goal=:test,))

@nanosoldier
Copy link
Collaborator

Your job submission was not accepted. Consult the server logs for more details (cc @maleadt).

@JuliaLang JuliaLang deleted a comment from nanosoldier Sep 13, 2025
@DilumAluthge
Copy link
Member Author

@nanosoldier runtests(["DataFrames", "CSV"])

@DilumAluthge
Copy link
Member Author

DilumAluthge commented Sep 14, 2025

It looks like it only tests that it can load Pkg.

This is frustrating, because I cannot reproduce when I run PkgEval locally.

I tested locally with PkgEval#master1. Here is my script I ran:

using PkgEval

config = Configuration()

packages = [
    Package(; name="LinearAlgebra"),
    Package(; name="Pkg"),
]

df = evaluate([config], packages)

When I run locally, PkgEval runs the full test suite for the stdlibs that I tried (Pkg and LinearAlgebra).

Footnotes

  1. Technically, I used https://github.com/JuliaCI/PkgEval.jl/pull/287, because I was running on a RHEL derivative. I don't think that's relevant to the issue I'm seeing.

@nanosoldier
Copy link
Collaborator

The package evaluation job you requested has completed - no new issues were detected.
The full report is available.

Report summary

✖ Packages that failed

1 packages failed on the previous version too.

~ Packages that at least loaded

1 packages successfully loaded on the previous version too.

@DilumAluthge
Copy link
Member Author

Okay, so this is weird - the problem is happening with CSV.jl too. For CSV.jl (log), it only loaded the package - it didn't run the tests. Why is that happening?

@DilumAluthge
Copy link
Member Author

DilumAluthge commented Sep 15, 2025

I wonder if the external stdlibs are getting blacklisted (at least the ones I've tried, Pkg and LinearAlgebra). And perhaps CSV.jl is blacklisted too.

Let me try running every stdlib. Surely not all of them have been blacklisted.

@DilumAluthge
Copy link
Member Author

@nanosoldier runtests(["ArgTools", "Artifacts", "Base64", "CRC32c", "Dates", "Distributed", "Downloads", "FileWatching", "Future", "InteractiveUtils", "LazyArtifacts", "LibCURL", "LibGit2", "Libdl", "LinearAlgebra", "Logging", "Markdown", "Mmap", "NetworkOptions", "Pkg", "Printf", "Profile", "REPL", "Random", "SHA", "Serialization", "SharedArrays", "Sockets", "SparseArrays", "StyledStrings", "SuiteSparse", "TOML", "Tar", "Test", "UUIDs", "Unicode", "CSV", "DataFrames", "Example", "JSON", "Crayons"])

@nanosoldier
Copy link
Collaborator

The package evaluation job you requested has completed - no new issues were detected.
The full report is available.

Report summary

✖ Packages that failed

2 packages failed on the previous version too.

✔ Packages that passed tests

33 packages passed tests on the previous version too.

~ Packages that at least loaded

6 packages successfully loaded on the previous version too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Do not merge this PR!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants