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

versioninfo: add commit + commit-date #1324

Merged
merged 5 commits into from
May 13, 2022
Merged

versioninfo: add commit + commit-date #1324

merged 5 commits into from
May 13, 2022

Conversation

benlorenz
Copy link
Member

@benlorenz benlorenz commented May 12, 2022

commit and commit-date will only work when the package is used from a git clone. now works even with Pkg.jl clones using the cached repo.

add full option to enable all output and use it in the issue template.

full output looks like this:

julia> Oscar.versioninfo(full=true)
OSCAR version 0.9.0-DEV - #bl/versioninfo, 3abd56b310 -- 2022-05-12 23:36:56 +0200
  combining:
    AbstractAlgebra.jl   v0.25.3
    GAP.jl               v0.8.0-DEV - #bl/sigchld, 6e62777 -- 2022-04-14 13:20:04 +0200
    Hecke.jl             v0.13.4
    Nemo.jl              v0.30.0
    Polymake.jl          v0.7.2 - #master, 9242f08 -- 2022-04-23 23:39:19 +0200
    Singular.jl          v0.10.1
  building on:
    Antic_jll               v0.200.501+0
    Arb_jll                 v200.2200.0+0
    Calcium_jll             v0.400.102+0
    FLINT_jll               v200.800.500+0
    GAP_jll                 v400.1192.2+1
    Singular_jll            v403.1.300+0
    libpolymake_julia_jll   v0.8.0+2
    libsingular_julia_jll   v0.23.1+0
    msolve_jll              v0.2.3+1
    polymake_jll            v400.600.0+0
See `]st -m` for a full list of dependencies.

Julia Version 1.8.0-beta3
Commit 3e092a2521 (2022-03-29 15:42 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: 8 × 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-13.0.1 (ORCJIT, tigerlake)
  Threads: 1 on 8 virtual cores
Environment:
  JULIA_CXX_RTTI = 1
  JULIA_CPU_TARGET = native
Official https://julialang.org/ release

(Oscar is using a git directory, GAP as well, for Polymake.jl I did ]add Polymake#master.

@benlorenz benlorenz requested a review from lkastner May 12, 2022 20:41
@benlorenz benlorenz marked this pull request as ready for review May 13, 2022 09:06
Copy link
Member

@lkastner lkastner left a comment

Choose a reason for hiding this comment

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

I had thought about mentioning earlier versions of Oscar.versioninfo that do not provide the full param, but they have only been there for such a short time that it doesn't make sense. At least in my opinion, feel free to adjust differently.

Copy link
Member

@fingolfin fingolfin left a comment

Choose a reason for hiding this comment

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

Thank you

@benlorenz benlorenz merged commit aedef9d into master May 13, 2022
@benlorenz benlorenz deleted the bl/versioninfo branch May 13, 2022 14:18
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