Path variable management: KIGITHUB is empty but gray -> error on ok if kept empty -> warning if filled

Bug #1809300 reported by Rene Poeschl on 2018-12-20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Jeff Young

Bug Description

I observe a strange (and annoying) behavior with the path management dialogs.
They include the old KIGITHUB variable. however it is empty and dark gray (implying not set able by the user)

If i press ok without filling it i get an error "not allowed to leave variable empty"
If i fill it i get a warning "this path is set outside of kicad ..."

I am not sure if this is an issue of my setup, the installer or if it is an issue in kicad it self.

I am running ubuntu 16.04 (virtual machine)
With both kicad 5.0.2 and kicad nightly (both installed from the ppa)

5.0.2 also includes KIGITHUB as grayed out but it does not complain about it being empty (unless i try to edit it)

The kicad version:
Application: kicad
Version: 6.0.0-rc1-unknown-24f9bfa~84~ubuntu16.04.1, release build
    wxWidgets 3.0.2
    libcurl/7.47.0 OpenSSL/1.0.2g zlib/1.2.8 libidn/1.32 librtmp/2.3
Platform: Linux 4.15.0-43-generic x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.2 (wchar_t,wx containers,compatible with 2.8) GTK+ 2.24
    Boost: 1.58.0
    OpenCASCADE Community Edition: 6.8.0
    Curl: 7.47.0
    Compiler: GCC 5.4.0 with C++ ABI 1009

Build settings:

Jeff Young (jeyjey) wrote :

A dark gray variable means it's defined in your environment. Type 'env' into a shell windows and see what it reports. You should see KIGITHUB there, although finding where it's defined can be trickier.

I'll merge a fix shortly which turns off the warnings for externally-defined variables.

Changed in kicad:
assignee: nobody → Jeff Young (jeyjey)
importance: Undecided → Low
status: New → Confirmed
milestone: none → 5.1.0
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision f6e6ac3162e22d362bb120fdf8cef1f2e9832570

Changed in kicad:
status: Confirmed → Fix Committed
Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers