-
Notifications
You must be signed in to change notification settings - Fork 328
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
ptp2 driver does not support file downloads over 4GB #582
Comments
nikon dslr ... now can do > 4gb files. |
you can try if the filresizes are ok. downloading will not yet work. But for the Ricoh .. can you do |
can you also add the summary-debug.txt ? |
They aren't' OK yet for me: -#18 R0012112.MP4 rd 4194304 KB 3840x1920 video/mp4
+#18 R0012112.MP4 rd 4194304 KB 3840x1920 video/mp4 Should the results depend on whether I have https://github.com/libmtp/libmtp/pull/68/files in my udev rules (currently those are in for my machine)? |
no, its entirely gphoto2 side... can you attach debug output of list files gphoto2 -L --debug --debug-logfile=list-files.txt and attach list-files.txt |
ricoh-theta-v-list-files.txt The |
thanks! one thing, you need to update libgphoto2 to current GIT ... the fix is not in 2.5.26 release |
(version should report as 2.5.26.1) |
Ah, I do have multiple versions available as compile outputs. Here are the |
i added another smnall commit that might fix the fileisze if you want to try. |
Some more hardware tested: |
That seems to list correctly on commit 93f2b14:
So I guess now only the download part is missing? |
To test on my Galaxy S7, I just used |
sounds like a workable idea, will try :) |
actually the ptp_getobject code in libgphoto2 was already capable of downloading > 4GB files... I reenable that code... |
…on 2.5.28 Arkell Rasiah (2): camlibs/ptp2/ptp-pack.c: Fix get-config aperture for R5. camlibs/ptp2/ptp-pack.c: Qualify previous change only for olcver=0x13 cameras. Daniel Schulte (20): camlibs/ptp2: Always return after PTP/IP IO errors (#595) WIP: Make PTP/IP implementation work on Windows (#596) gp_camera_exit: Return result of camera exit function if possible camlibs/ptp2: Better cross-platform error handling camlibs/ptp2: fuji: Also close jpg socket if connection fails camlibs/ptp2: Better network timeout handling camlibs/ptp2: Add more error handling to camera_exit camlibs/ptp2: Add timeout support for connect/read/write socket operations camlibs/ptp2: Reduce log level of debug message camlibs/ptp2: Unpack Canon EOS changes for INT32 properties camlibs/ptp2: fuji: Fix property datatype in camera_trigger_capture camlibs/ptp2: fuji: Ignore added preview object while live-view is active camlibs/ptp2: Add Fuji priority mode setting camlibs/ptp2: fuji: Use PriorityMode define in camera (un)prepare code camlibs/ptp2: Fuji: Fix some PTP/IP error messages camlibs/ptp2: Fuji: camera_trigger_camera: Poll slower camlibs/ptp2: Don't send final ptp_initiatecapture with Fuji cameras camlibs/ptp2: Fuji PTP/IP init packet must have length 82 camlibs/ptp2: Fuji: When connecting via PTP/IP try to connect twice Add add missing ISO values from Canon EOS R6 David Rosca (3): ptp2: Disconnect PTP/IP connection in camera_exit filesys: Add gp_filesystem_set_info_dirty ptp2: Reload object info on PTP_CANON_EOS_CHANGES_TYPE_OBJECTINFO_CHANGE Gergely Erdélyi (1): Add model detection for Fujifilm X100V (#627) Hans Ulrich Niedermann (112): Fix typos found by codespell in *.m4 gitignore: Anchor patterns at top_srcdir gitignore: Add /tmpwrk*/ Fix grammar and spelling in comments and text Remove obsolete gp2ddb (gphoto2 device database) idea configure.ac: Avoid po-directories error Use AC_SEARCH_LIBS to find where mmap(2) is defined gh workflows: remove requirements for flex, bison INSTALL: add gtk-doc Debian package name Document using GP_CAMLIB_SET on make command line Run libgphoto2_port tests only AFTER building iolibs Leave comment in Makefile.am about subdir sequence gitignore: Add a few forgotten files for in-source-tree builds Remove remnants of OS/2 support use the proper word fix broken check of CPP macro definition after #elif Terminate assignment with semicolon instead of comma Fix parameters being marked unused when they are being used Use proper format string for FDHANDLE type comment out unused constant/variable definition Use consistently named non-reserved CPP macros for header files Use a single definition of BayerTile enum utf-8: Convert ISO-8859 text to UTF-8 utf-8: change xml/html from iso-8859-1 to utf-8 utf-8: fix weird minus/hyphen character utf-8: convert old camlibs/**/*.txt files from latin1 Remove cvs-tags make target HACKING.md: Update include guard macro name Use same autoconf requirement in libgphoto2_port Group iolib list related definitions Avoid accidental include file name collisions Rename bayer.[ch] to stv0680-bayer.[ch] Use common code for defining iolibdir and camlibdir Make CAMLIBS and IOLIBS fallbacks configurable Replace --with-foodir= argument by foodir= variable Fix check output for DEFAULT_CAMLIBS and DEFAULT_IOLIBS test-camera-list: Add --help output test-camera-list: Widen column for 4 digit number of camlibs Remove unused remnants of convenience libltdl Sync gp-byteorder.m4, supporting 64bit values Use && and || instead of test arguments -a and -o Remove rpm packaging Remove unused code adding x bit to *.sh Remove linux-hotplug support Update libphoto2_port/po/de.po to UTF-8 after all NEWS: converted text from iso to utf-8, removed linux-hotplug Typo fix: It is _SOURCES, not _SOURCE example: include <unistd.h> for close(2), unlink(2), getpid(2) tp6081-dump: include <unistd.h> for close(2), add more static Here document: Use <<EOF everywhere sync gp-check-library.m4 between gphoto-m4 dirs Factor out i18n stuff into separate i18n.h files Delete old debug comments library versioning comment without trailing spaces examples: Force #include beginning with "libgphoto2/" Rename libgphoto2_port test directory tests POTFILES.in: Add ptpip, vusb iolibs POTFILES.in: Add remaining source files using i18n.h tests: include system headers before gphoto2 headers tests: declare local variable local sync gp-gettext-hack.m4 files between gphoto-m4 dirs HACKING.md: Misc small improvements (grammar, whitespace, etc.) HACKING.md: Update with C99 being our requirement HACKING.md: Add note about printf(3) fmt strings using "%m" gp-camlibs.m4: Fix GP_SLEEP invocation in in Gentoo hack tests: Remove old pedantic compilation tests tests: Always define, but only test conditionally tests: add pedantic compile tests for -std={c2x,c++20} Make the compiler warnings work with clang, gcc, and others Update libltdl checks (GP_LIBLTDL) Update the checks for libjpeg (GP_LIBJPEG) gitignore for tests/ Use m4_pattern_forbid to find unnoticed GP_ strings AC_INIT change to arguments on multiple lines AM_INIT_AUTOMAKE change to indented multi-line form AC_CONFIG_FILES change to indented multi-line form m4 quote all macro arguments gh workflow: install libgd-dev Consistent checks for asm .symver and version-script Use the same gettext setup Use LINGUAS files instead of ALL_LINGUAS variable remove old libtool/libltdl comments make gphoto-m4 directories identical indentation for readability Remove definition of localedir Add build tool version comment Require at least autoconf-2.69 Require at least automake 1.14 Only use a single gphoto-m4/ subdir Add "mode: autoconf" line to all *.m4 files with-camlibs: grammar fix for standard set description with-camlibs: document "everything" set in the --help output Use non-reserved macro name for generated gphoto-endian.h Add compiletime-assert.h for COMPILETIME_ASSERT() macro ptp2/ptp.c: tell emacs the c-file-style ptp2/ptp.c: Rewrite PTP_CNT_INIT to build with pure C99 remove whitespace around AS_HELP_STRING options Remove --with-doc-dir and --with-html-dir rearrange test-pedantic definitions The pedantic compilation flags might ignore some warnings RELEASE-HOWTO.md: Change the name of the IRC network Invoke AC_PROG_SED before using ${SED} Require at least libtool 2.4.2, use LT_INIT Remove commented out hack for pre-2001 gettext Add TODO comment about possibly using gettext's gettext.h MAINTAINERS: remove rpm builds Document translating languages outside Translation Project RELEASE-HOWTO.md: Improve markdown readability Require gettext >= 0.18.3; remove GP_GETTEXT_HACK Add pull-translations target Add test-i18n.c i18n test program Verbose gettext domain consistency checks Ingvar Stepanyan (11): Provide correct search size to memchr Revamp Sony ISO logic Fix infinite loop when setting Sony values Fix overshoot logic for Sony enums Mark properties without setters as readonly Add file extension for Sony ARW files Remove duplicate GP_MIME_ARW Normalize Sony shutter speeds Fix memory leak in capture preview for Sony Add Sony a6600 descriptors Add few more movie modes for Sony Jakub Bogusz (1): updated polish Jan Kundrát (4): camlibs/ptp2/ptp-pack.c: Fix aperture on EOS RP Canon: basic control over Movie Servo AF Add mode dial positions available on EOS RP or 5Div Add HDR exposure mode from EOS RP Jan Tojnar (1): print-camera-list: clarify help Jaroslav Škarvada (1): Fixed build in mingw Jiri Machalek (1): mime type update for CR3 Marcus Meissner (242): 2.5.26.1 development starts define _DARWIN_C_SOURCE for strdup on MacOS. fixes gphoto/libgphoto2#552 also do tar.xz do not add changecamera mode on the Nikon V1 do not query for PTP_DPC_NIKON_LiveViewStatus mark nikon v1 as with broken cpature,. also disable the 90c7 event handling as this also crahes the camera. gphoto/libgphoto2#569 on nikon v1, also blank out the nikon specific capture pass the usb interface to the control pipeline command functions added 2 new ricoh models from libmtp/libmtp#68 comment out the broken flags again so we get good filesiszes from Samsung Galaxy dedup a name revert back to 0 byte PTP strings turning into "" instead of NULL. detect objects larger than 4gb. added 64bit file support for Nikon DSLR add 4gb news added ricoh theta z1 and v debugging wait busy also handle the bulb captuzre busyness actually use the proplist for the size ... gphoto/libgphoto2#582 remove () check for -lws2_32 to enable winsock support for ptpip on windows fix jpeg -> ws232 typo added fuji finepix s8600 do not error on files larger than 4GB, our ptp getobject method can handle them. updated added new changeapplicationmode for Z6 II added more z6 ii commands decode new ptp oc nikon codes added P1000 added liveview zoom area ... left hand value of WidthxHeight added fuji x-s10 usb ids and debug output according to http://fujifilm-x.com/global/support/compatibility/software/x-webcam/ this is doing capture preview at least, gphoto/libgphoto2#603 fuji x-s10 in webcam mode adjust busy wait on fuji preview capture to be 5ms instead of 1ms added g12 ids added 2 more nikon event codes add a workraound to detect card value even if there is no enum. added EOS M1 Mark II another ID, from timelapse-VIEW added Sony A7S III for Sony ALPHA, return a static list of f-numbers if we get "range" for it. the Nikon Z series movie capture want to wait for the event confirming finished movie capture before proceeding add event queue check / remove by type functions. switch to use a remove event by type functiion in nikon capture reduce wait times in the eos preview loop again. added powershot g12 debug decode EOS names for easier debugging. the getviewfinderdata has 3 arguments... also use 00200000 like in current traces implemented nikon check event ex added missing ptp-pack part for nikon_check_event_ex handle canon ptpip same as nikon ptpip add eventmode option for canon eos. hook in nikon geteventex function. support more MTP ObjectPropDesc types The EOS M6 Mark II is more similar to the traditional EOS, than the EOS M series. x-s10 has only webcam mode, no capture mode... fixes gphoto/libgphoto2#603 added fuji x-s10 iopcodes year is 2021 autopoweroff might be 32bit refine debug of ptpip responses skip the unknown data in front of the preview using the 4 byte offset added sony a7r2 debug handle deviceinfo changed events in wait_for_event remove superflous print device info added streamid parameter to GetStreamInfo added canon eos m200 blank serial nr added 9414 getliveview config added more autoexposuremodes (from eos m6 mark 2) decode more canon events a bit better set the cature timeout BEFORE calling captureimage, helps nikon v1 generic initiate capture path: manage all Z series with te z6 config report movieprohibit and liveviewprohibit camlibs/ptp2/ptp-pack.c: Fix get-config iso/aperture for R5. (#621) seperate prohibits by comma move some enums to their known places, eos cameranickname is a string report camera nickname added OLC 0x12 for EOS M6 Mark2 offsets adjust more offset for eos m6 mark 2 OLC added strobofiring, flashcharged, oneshotrawon, autoexposuremodedial return the correct OK value on success added liveviewsize configuration for panasonic. set is not workign fully. gphoto/libgphoto2#575 also include x value in config for now replaced some malloc(x*y) by calloc(x,y), also removed then superflous memset replaced some malloc(x*y) by calloc(x,y), removed 1 superflous memset replaced more malloc(x*y) by calloc(x,y) move around panasonic liveviewdata use define instead of magic constant optimize panasonic whitebalance added more info on panasonic things implemented getcapturetarget add more defines for panasonic, use some defines for queries instead of magic values added movie recording for panasonic lumix added panasonic deviceprop sniffs added more sniffed data added a missing propdesc freeing added missing propdescfree added some missing free propdesc and free objectinfo in sony trigger capture movie is a toggle,. not text removed a double free (Coverity) imported from libmtp removed a superflous argument (Coverity) updated to current state fixed a memleak (Coverity) fixed incorrect NULL check (Coverity) 2.5.27 release update RELEASE-HOWTO.md 2.5.27.1 dev release added Konica-Minolta:DiMAGE Z6 move Fujifilm X100V to corect ordered by place olympus would wait twice the time due to a missing return added polympus note also added DSC-RX0M2 here for intiial wait, see gphoto/gphoto2#349 added note about manual focusing not needing delay added sony a7c also wait 3 seconds for the A7C add download when announcing added #define _DARWIN_C_SOURCE as in other files to avoid implicit use isoc99 source gphoto/libgphoto2#633 use darwin_c_source ... isoc99 source does not seem to cut it for macos. gphoto/libgphoto2#633 support sony prioritymode, init it to 1. added d780 aapture settings. added coolpix b500 added movieexposurecompensation fix movieexposurebiascompensation added fuji xpro3 start sigma fp support. gphoto/libgphoto2#537 added sigmafp liveview function, decode sigmafp codes sigma fp liveview now works first sucessfull (but not fully correct) capture code for sigma fp added cleardbsingleimage after capture implemented getpictfileinfo2ext for sigma fp implemented capture status, used it to cehck for finished capture wait a bit longer for image processing add some more defines, name 9035 camerainfo pass id to clearimagedb debug output all data groups for the sigma fp start the work of allowing sigma fp configuration first try at shutterspeed for sigma fp added sonx rx1r m2 added eos m50m2 added sigma fp shutterspeed mapping from SDK table added sigma fp aperture handling (untested) added correct property for manualfocus sony added liveview support to x-t30, but no capture. gphoto/gphoto2#196 added d5000 report that nikons cannot upload fuji: instead of special casing fuji, just return GP_OK from the fuji branch added Fuji:Fujifilm GFX 100S updated NEWS added eos1300d allow cancel for bytes_read >= 1MB to cover blob read size in ptp added a7s data harmoize the jpeg extractors pointer arithmetic for memchr to be valid C use the large jump on first try, fall back to single jumps on next loops added more braces to the new expressions to make clear what we test make the textual description of syncdatetimeutc also mention UTC fixed some memory leaks caused by ptp_getobjectinfo imbalanced with ptp_free_objectinfo, see gphoto/libgphoto2#690 temporary allow setting all properties on sony, as ourdetection logic does not seem to be right. remove memory leaks report changed config labels during wait decode value of regular widgets added property change decoding to EOS and Olympus more debug, also switch conditions to avoid O(n2) complexity srtore DevPropCode, needed by current config name matcher trying to fix eos reader emit also new values in the config update work (draft work) gphoto/libgphoto2#677 factorize out the jpeg extraction out of data blobs further factor out the preview saving and meta data setting, as its common use generic preview.jpg name, not sony for ... fuji move another preview jpeg extractor to factorized function factor out regular image additions repeated code added fuji gfx100 second id reported via SF for sony preview capture use a timeout of 1 second replaced R2 -> RP added sony zv-e10 added sony zv e10 pass lenght of datablobs into the directory compare function to avoid overreads harden the directory compare function against overreads handle busy errors on getobjectinfo turn zoom widget into a range. check for errors from widget functions merged music-players.h from libmtp also register port vusb: start a sample for libfuzz addred a missing gp_port_info_new remove 1 warning added sample-libfuzz ... semi-uable for libfuzzer added comment how to build fuzzer, and current problems fixed uninitialized memory access / potential crash better comments added u1-u3 for nikon decode usermode added z6 note report file info for debugging report only if flag set also handle j1 like v1 ...blank out getevent and imagerecinsdram report file info requzest the objectinfo to be loaded fixed return code from unprepare capture during EOS exit emit only matching resetuilock if we did setuilock added j1 issue fix build after bayer.h change added pentax k3 II in ptzp mode only ovbrwrite the PEF header for the K100D. emit a FILE CHANGED event if the ObjectInfo changes. some updates somre more updates rev the age of the library. added note for the 1:0:7 rev add more news make update-po updated german german translation completed spelling fixes more speling fixes more spellchecking avoid duplicate mvoie / liveview prohibition code strings for translation simplified after prohibit simplification added creative auto mode dial setting sync from libmtp handle motorola as MTP... fixes gphoto/gphoto2#463 we need to refetch the ptp object after get_folder_from_handle, as that call might invalidatze the poionter Factored out more things into add_object_to_fs_and_path helper factory out 2 more instances for standardized object adding factor out more instances of 'add handle to filesystem' updated de fixed an uninit variable added c++ wrappers remove superflous ; avoid crash on freed memory added pentax k1 added pentax k1 aded sony sx-hx80 added pentax k1 id added Sony ILCE-7RM4A gphoto/libgphoto2#749 remove dup k1 adjust hex id to have 4 digits, fixes gphoto/libgphoto2#753 fix camera name, fixes gphoto/libgphoto2#754 sync from libmtp added more NEWS keep canon eos remotemode=1 on exit, to keep the display on 2.5.28 release Ole Dalgaard (1): Add Nikon Zfc support Ondrej Zary (3): sierra: Add workaround for Epson PhotoPC 500 at higher speeds sierra: Add SIERRA_PACKET_CANCEL (0x18) as valid resonse sierra: Add curly braces to improve code readability Pino Toscano (2): directory: use statvfs instead of statfs directory: enable storage_info_func on any platform Rafał Hirsz (1): Add _DARWIN_C_SOURCE to most files that use string.h (#615) Stéphane Aulery (2): updated french updated french Sven Killig (1): added PowerShot exposure compensation values Wolfram Sang (1): ptp2: add Canon IXUS 132 Yuri Chornoivan (1): updated ukrainian Zeranoe (2): Remove stdint.h shim (#625) Fix byte swap calls (#626) avisecag (1): Added USB ID for Nikon Z6_2 (#583) fahrion2600 (2): Recognize Nikon Z 7II and fix VendorExtensionID over PTP/IP (#605) Also set params->controlmode in _put_Nikon_ControlMode (#606) gregmag (1): Update library.c tomsch1 (2): Update library.c Create sony-a7r3a.txt trilader (1): Add support for reading Canon EOS MovieMode switch and setting it on/off
Describe the bug
Currently we do not support file download over 4GB in the ptp driver.
Largely because these are special handled by ptp, and Marcus did not yet find a device for testing. :(
Name the camera
e.g. Ricoh Theta supposedly allow such files, also Android phones , also Fuji XT?
The text was updated successfully, but these errors were encountered: