Skip to content

Fixed capture-preview naming behaviour#3

Merged
msmeissn merged 1 commit intogphoto:masterfrom
jabiinfante:master
Sep 20, 2015
Merged

Fixed capture-preview naming behaviour#3
msmeissn merged 1 commit intogphoto:masterfrom
jabiinfante:master

Conversation

@jabiinfante
Copy link
Copy Markdown
Contributor

Launching gphoto2 as follows:

// And invoke capture-preview, saves the picture using "--filename" argument
gphoto2: {/tmp/pics} /> capture-preview
Saving file as pic.jpg

The prefix "thumb_" should be added in front of the "--filename" specified.

Without "--filename" argument:
cagphoto2: {/tmp/pics} /> capture-preview
Saving file as thumb_capture_preview.jpg

With "--filename" argument:
gphoto2: {/tmp/pics} /> capture-preview
Saving file as thumb_pic.jpg

Launching gphoto2 as follows:

// And invoke capture-preview, saves the picture using "--filename" argument
gphoto2: {/tmp/pics} /> capture-preview
Saving file as pic.jpg

The prefix "thumb_" should be added in front of the "--filename" specified.

Without "--filename" argument:
cagphoto2: {/tmp/pics} /> capture-preview
Saving file as thumb_capture_preview.jpg

With "--filename" argument:
gphoto2: {/tmp/pics} /> capture-preview
Saving file as thumb_pic.jpg
@jabiinfante
Copy link
Copy Markdown
Contributor Author

You are right, but i think the mistake might be over here:

https://github.com/gphoto/libgphoto2/blob/master/libgphoto2/gphoto2-file.c#L919

The patch is only applying GP_FILE_TYPE_PREVIEW for pics taken with "capture-preview". I think then "thumb_" naming convetion should be fixed on libgphoto2... right?

msmeissn added a commit that referenced this pull request Sep 20, 2015
Fixed capture-preview naming behaviour
@msmeissn msmeissn merged commit 1dcc035 into gphoto:master Sep 20, 2015
@msmeissn
Copy link
Copy Markdown
Contributor

I will see if I can do it like this:

  • no filename set, name it capture_preview.jpg always
  • filename set , use thumb_ logic

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.

2 participants