Commit f23e151b907 breaks compilation on Windows

Bug #1854562 reported by jean-pierre charras
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Committed
Critical
Seth Hillbrand

Bug Description

Commit f23e151b907 (Use wxArrayString instead of vector<wxString> in BUS_ALIAS) breaks compilation (at least on msys2) each time the code try to copy a wxArrayString into a std::vector<wxString>.

The error message is:
F:/kicad-launchpad/git_testing/eeschema/netlist_object.cpp: In member function 'void NETLIST_OBJECT::ConvertBusToNetListItems(NETLIST_OBJECT_LIST&)':
F:/kicad-launchpad/git_testing/eeschema/netlist_object.cpp:254:47: error: no match for 'operator=' (operand types are 'std::vector<wxString>' and 'wxArrayString')
  254 | bus_contents_vec = alias->Members();
      | ^
In file included from C:/msys32/mingw32/include/c++/9.2.0/vector:72,
                 from D:/wxWidgets-3.1.3/include/wx/arrstr.h:19,
                 from D:/wxWidgets-3.1.3/include/wx/wx.h:21,
                 from F:/kicad-launchpad/git_testing/include/fctsys.h:28,
                 from F:/kicad-launchpad/git_testing/eeschema/netlist_object.cpp:31:
C:/msys32/mingw32/include/c++/9.2.0/bits/vector.tcc:198:5: note: candidate: 'std::vector<_Tp, _Alloc>& std::vector<_Tp, _Alloc>::operator=(const std::vector<_Tp, _Alloc>&) [with _Tp = wxString; _Alloc = std::allocator<wxString>]'
  198 | vector<_Tp, _Alloc>::
      | ^~~~~~~~~~~~~~~~~~~
C:/msys32/mingw32/include/c++/9.2.0/bits/vector.tcc:199:42: note: no known conversion for argument 1 from 'wxArrayString' to 'const std::vector<wxString>&'
  199 | operator=(const vector<_Tp, _Alloc>& __x)

Tags: eeschema
Changed in kicad:
milestone: none → 6.0.0-rc1
Revision history for this message
Wayne Stambaugh (stambaughw) wrote :

It also breaks linux builds.

Changed in kicad:
status: New → Triaged
importance: Undecided → Critical
Revision history for this message
Wayne Stambaugh (stambaughw) wrote :

@Jon, when making changes to fix build errors on VS, please check that the changes build at a minimum on linux. How soon can you get around to fixing this? I was planning on disabling the launchpad repo this morning.

Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision d20d310fccdbbc65cca2da63f58cede4fba3a357
https://git.launchpad.net/kicad/patch/?id=d20d310fccdbbc65cca2da63f58cede4fba3a357

Changed in kicad:
status: Triaged → Fix Committed
assignee: nobody → Seth Hillbrand (sethh)
Revision history for this message
Seth Hillbrand (sethh) wrote :

Jon, feel free to modify this fix if you have a better one in mind

Revision history for this message
Jon Evans (craftyjon) wrote :

Sorry, I did not see this until just now. Thanks for fixing it, Seth

tags: added: eeschema
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.