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

kicad: update to 6.0.1 #34917

Closed
wants to merge 8 commits into from
Closed

kicad: update to 6.0.1 #34917

wants to merge 8 commits into from

Conversation

PoroCYon
Copy link
Contributor

@PoroCYon PoroCYon commented Jan 7, 2022

Testing the changes

  • I tested the changes in this PR: YES/briefly: I opened a project of mine, looked around and made some minor edits, tried the main components (schematic editor, PCB editor, 3D viewer), but I didn't test all features (scripting, gerber output with weird contents, SPICE integration, ...).

[ci skip] because kicad-packages3D is humongous, and KiCAD itself takes a long while to build, and all other packages are trivial (only assets).

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc
  • I did not build this PR locally for these architectures as I don't have any of these machines (thus cannot test it) and cross-compiling would take too long.

So, preferably, a few other people should test this PR as well.

@unspecd
Copy link
Contributor

unspecd commented Jan 7, 2022

  1. It works for me (on x86_64-musl), but one test failed:
=> kicad-6.0.0_1: running do_check ...
[0/1] Running tests...
Test project /builddir/kicad-6.0.0/build
    Start 1: qa_python
1/8 Test #1: qa_python ........................   Passed   63.83 sec
    Start 2: qa_common
2/8 Test #2: qa_common ........................   Passed    2.34 sec
    Start 3: qa_gerbview
3/8 Test #3: qa_gerbview ......................   Passed   21.09 sec
    Start 4: qa_eeschema
4/8 Test #4: qa_eeschema ......................***Failed   15.42 sec
Running 62 test cases...
../qa/eeschema/test_ee_item.cpp(124): error: in "EeItem/Rotate": check aItem->GetBoundingBox().GetTop() == aOriginalItem->GetBoundingBox().GetTop() has failed [-529945 != -529188]
Failure occurred in a following context:
    Class: SCH_SHEET
    Position: -1000000 -500000, Reference: -1000000 -500000
../qa/eeschema/test_ee_item.cpp(128): error: in "EeItem/Rotate": check aItem->GetBoundingBox().GetBottom() == aOriginalItem->GetBoundingBox().GetBottom() has failed [-433225 != -433982]
Failure occurred in a following context:
    Class: SCH_SHEET
    Position: -1000000 -500000, Reference: -1000000 -500000
../qa/eeschema/test_ee_item.cpp(124): error: in "EeItem/Rotate": check aItem->GetBoundingBox().GetTop() == aOriginalItem->GetBoundingBox().GetTop() has failed [-529945 != -529188]
Failure occurred in a following context:
    Class: SCH_SHEET
    Position: -1000000 -500000, Reference: -1000000 0
../qa/eeschema/test_ee_item.cpp(128): error: in "EeItem/Rotate": check aItem->GetBoundingBox().GetBottom() == aOriginalItem->GetBoundingBox().GetBottom() has failed [-433225 != -433982]
Failure occurred in a following context:
    Class: SCH_SHEET
    Position: -1000000 -500000, Reference: -1000000 0

[etc...]

*** 162 failures are detected in the test module "Common Eeschema module tests"


    Start 5: qa_sexpr
5/8 Test #5: qa_sexpr .........................   Passed    0.13 sec
    Start 6: qa_kimath
6/8 Test #6: qa_kimath ........................   Passed    0.92 sec
    Start 7: qa_pcbnew
7/8 Test #7: qa_pcbnew ........................   Passed  125.88 sec
    Start 8: qa_kicad2step
8/8 Test #8: qa_kicad2step ....................   Passed    4.92 sec

88% tests passed, 1 tests failed out of 8

Total Test time (real) = 234.73 sec

The following tests FAILED:
	  4 - qa_eeschema (Failed)
Errors while running CTest
FAILED: CMakeFiles/test.util 
cd /builddir/kicad-6.0.0/build && /usr/bin/ctest --force-new-ctest-process
ninja: build stopped: subcommand failed.
  1. You could remove unused configure_args:
  2. Not sure, but instead of deleting disable-warnings.patch, you could update it:
--- a/CMakeModules/Warnings.cmake
+++ b/CMakeModules/Warnings.cmake
@@ -38,13 +38,6 @@
     set( WARN_FLAGS_CXX "-Wall" )
 
 
-    # Warn about missing override specifiers
-    CHECK_CXX_COMPILER_FLAG( "-Wsuggest-override" COMPILER_SUPPORTS_WSUGGEST_OVERRIDE )
-
-    if( COMPILER_SUPPORTS_WSUGGEST_OVERRIDE )
-        set( WARN_FLAGS_CXX "${WARN_FLAGS_CXX} -Wsuggest-override" )
-        message( STATUS "Enabling warning -Wsuggest-override" )
-    endif()

@PoroCYon
Copy link
Contributor Author

PoroCYon commented Jan 7, 2022

Thanks for the feedback, I've incorporated your suggestions. However, I can't seem to replicate the test failure:

=> kicad-6.0.0_1: running do_check ...
[0/1] Running tests...
Test project /builddir/kicad-6.0.0/build
    Start 1: qa_python
1/8 Test #1: qa_python ........................   Passed    1.37 sec
    Start 2: qa_common
2/8 Test #2: qa_common ........................   Passed    0.04 sec
    Start 3: qa_gerbview
3/8 Test #3: qa_gerbview ......................   Passed    0.11 sec
    Start 4: qa_eeschema
4/8 Test #4: qa_eeschema ......................   Passed    0.83 sec
    Start 5: qa_sexpr
5/8 Test #5: qa_sexpr .........................   Passed    0.01 sec
    Start 6: qa_kimath
6/8 Test #6: qa_kimath ........................   Passed    0.02 sec
    Start 7: qa_pcbnew
7/8 Test #7: qa_pcbnew ........................   Passed   45.37 sec
    Start 8: qa_kicad2step
8/8 Test #8: qa_kicad2step ....................   Passed    0.17 sec

100% tests passed, 0 tests failed out of 8

Total Test time (real) =  47.93 sec

Maybe it's a glibc vs musl difference? I forgot to mention I'm using glibc. Either way, I'd need some help to fix this as I can't replicate it.

@lemmi
Copy link
Member

lemmi commented Jan 16, 2022

6.0.1 is out. Tested this briefly on x86_64-glibc with old boards.

@PoroCYon
Copy link
Contributor Author

PoroCYon commented Jan 16, 2022

Thanks for the notice, I've update the templates to 6.0.1. kicad-doc and kicad-i18n don't seem to have downloads anymore, so I've removed the packages.

EDIT: ran the tests again, all seem to pass

@PoroCYon PoroCYon changed the title kicad: update to 6.0.0 kicad: update to 6.0.1 Jan 16, 2022
Copy link
Member

@ericonr ericonr left a comment

Choose a reason for hiding this comment

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

These should be separate commits for each template.

kicad should get a replaces="kicad-i18n>=0" line, since translations are now shipped there instead of in a separate repo.

I'm looking into kicad-doc as well now.

srcpkgs/kicad/template Show resolved Hide resolved
srcpkgs/kicad/template Outdated Show resolved Hide resolved
srcpkgs/kicad/template Outdated Show resolved Hide resolved
@PoroCYon
Copy link
Contributor Author

I've split the commits into separate ones for all the packages, and integrated your feedback.

@PoroCYon
Copy link
Contributor Author

Do you have any news on the kicad-doc situation?

@foopub
Copy link
Contributor

foopub commented Jan 26, 2022

Do you have any news on the kicad-doc situation?

What's the issue? https://kicad-downloads.s3.cern.ch/docs/kicad-doc-6.0.1.tar.gz works for me.

Also here's the checksum
sha256sum Downloads/kicad-doc-6.0.1.tar.gz
6093c99cfde827fb42fc351ef301f0dcad805f0c256ef02a0e82d9daafc238d3 Downloads/kicad-doc-6.0.1.tar.gz

@PoroCYon
Copy link
Contributor Author

PoroCYon commented Jan 27, 2022

Ah, the 6.0.0 link was 404'ing for 6.0.0 when I tried it. I'll readd the package tomorrow.

ericonr pushed a commit to ericonr/void-packages that referenced this pull request Jan 28, 2022
@ericonr ericonr closed this in 7b6227d Jan 28, 2022
@ericonr
Copy link
Member

ericonr commented Jan 28, 2022

For next time, it'd be best to just let CI run. See bb4b6fb

@foopub
Copy link
Contributor

foopub commented Jan 29, 2022

For next time, it'd be best to just let CI run. See bb4b6fb

Seems CI builds fail for all but x86_64 glibc... https://github.com/void-linux/void-packages/actions/runs/1736427514
I tried looking through the build logs but they're too long to view online. Tried to download, but it reached >100mb before I cancelled. Not sure what the issues are, and kicad is the most annoying program to debug in my experience (no runtime logs!) so I gave up at that point.

ElDifinitivo pushed a commit to ElDifinitivo/void-packages that referenced this pull request Feb 13, 2022
ElDifinitivo pushed a commit to ElDifinitivo/void-packages that referenced this pull request Feb 14, 2022
ElDifinitivo pushed a commit to ElDifinitivo/void-packages that referenced this pull request Feb 14, 2022
maciozo pushed a commit to maciozo/void-packages that referenced this pull request Feb 19, 2022
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.

None yet

5 participants