search

gphoto / gphoto2

The gphoto2 commandline tool for accessing and controlling digital cameras.
GNU General Public License v2.0
718 stars 116 forks source link

Incorrect Radio options fetched in shell mode #317

Open ManjunathaN opened 4 years ago

ManjunathaN commented 4 years ago

Describe the bug

Config like shutterspeed are dependent on other configs like autoexposuremode, and the radio choices should change accordingly. This options returned on gphoto2 --shell are incorrect.

Name the camera USB PTP Class Camera usb:002,116 Manufacturer: Canon.Inc Model: Canon EOS RP Version: 3-1.4.0

libgphoto2 and gphoto2 version gphoto2 2.5.23 gcc, popt(m), exif, no cdk, no aa, jpeg, no readline libgphoto2 2.5.23 all camlibs, gcc, ltdl, EXIF libgphoto2_port 0.12.0 iolibs: disk ptpip serial usb1 usbdiskdirect usbscsi, gcc, ltdl, USB, serial without locking

To Reproduce Steps to reproduce the behavior:

gphoto2 --set-config autoexposuremode=TV; gphoto2 --get-config=shutterspeed; gphoto2 --set-config autoexposuremode=P; gphoto2 --get-config=shutterspeed Works fine, returns only one option. Screen Shot 2020-05-10 at 11 45 09 AM

However, on gphoto2 --shell set-config-value autoexposuremode=TV get-config shutterspeed set-config-value autoexposuremode=P get-config shutterspeed Screen Shot 2020-05-10 at 11 45 41 AM

This options includes the options of both TV and P.

msmeissn commented 4 years ago

shutterspeed will be reported incorrectly on the EOS R currently, either update to current libfgphoto2 GIT or wait for 2.5.25 release in the near future.

msmeissn commented 4 years ago

Hmm, this is something else ...

Not sure what you mean should happen, the shutterspeed list of options should change in shell mode?

ManjunathaN commented 4 years ago

The list is changing in shell mode, but the caveat is it's not the correct list, as it includes options from both autoexposuremode - TV and P. Ideally, it should have listed the below options - Screen Shot 2020-05-10 at 11 45 09 AM

Moreover, subsequent set-config on shutterspeed fails. Is there an workaround to get the correct options?