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

.ci-output.py script is broken #15

Open
ajelinski opened this issue Feb 15, 2021 · 1 comment
Open

.ci-output.py script is broken #15

ajelinski opened this issue Feb 15, 2021 · 1 comment

Comments

@ajelinski
Copy link
Contributor

Using the script will be disabled because currently it does more harm than good.

This script was intended to make log analyzing easier. Unfortunately in many cases it hides crucial information from being printed in the log, e.g. https://github.com/litex-hub/litex-conda-eda/pull/63/checks?check_run_id=1901771647#step:3:4365

  
  .
      |
  
  .
      |                  ^^^^^^^^^^^^^^
  
  .
     Compiling atty v0.2.14
     Compiling cast v0.2.3
  
  ..
  Some errors have detailed explanations: E0034, E0308.
  For more information about an error, try `rustc --explain E0034`.
  
  ..
  To learn more, run the command again with --verbose.
  
  ..
  Traceback (most recent call last):
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/bin/conda-build", line 11, in <module>
      sys.exit(main())
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 474, in main
      execute(sys.argv[1:])
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 465, in execute
      verify=args.verify, variants=args.variants)
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/api.py", line 195, in build
      variants=variants
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/build.py", line 3005, in build_tree
      notest=notest,
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/build.py", line 2126, in build
      cwd=src_dir, stats=build_stats)
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/utils.py", line 410, in check_call_env
      return _func_defaulting_env_to_os_environ('call', *popenargs, **kwargs)
    File "/home/runner/work/litex-conda-eda/litex-conda-eda/workdir/conda-env/lib/python3.7/site-packages/conda_build/utils.py", line 390, in _func_defaulting_env_to_os_environ
      raise subprocess.CalledProcessError(proc.returncode, _args)
ajelinski added a commit that referenced this issue Feb 15, 2021
Currently `.ci-output.py` script causes more harm than good. It
prohibits many crucial lines from being printed in logs. For details see
issue #15.

Therefore it shouldn't be used to call `conda-build` until it's fixed.
@umarcor
Copy link
Member

umarcor commented Feb 19, 2021

Should this be closed since 885276d was pushed already?

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

No branches or pull requests

2 participants