Skip to content
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

Update timelapse.md #269

Closed
wants to merge 2 commits into from
Closed

Update timelapse.md #269

wants to merge 2 commits into from

Conversation

maglub
Copy link

@maglub maglub commented Oct 19, 2015

With the new release of Raspbian, some packages are deprecated, replaced by newer ones.

With the new release of Raspbian, some packages are deprecated, replaced by newer ones.
@XECDesign
Copy link
Contributor

Looks good, but seems to cut off abruptly at "Now run the following command:"

@JamesH65
Copy link
Contributor

It's always bugged me - the raspistill software already has a timelapse
mode that is easier to use than this. But I've never got round to writing
the documentation...

On 19 October 2015 at 13:51, XECDesign notifications@github.com wrote:

Looks good, but seems to cut off abruptly at "Now run the following
command:"


Reply to this email directly or view it on GitHub
#269 (comment)
.

@maglub
Copy link
Author

maglub commented Oct 19, 2015

Ops, sorry. It is abruptly cut off.

@bennuttall
Copy link
Contributor

Rather than Jessie a separate section, it should be the new default. The package names should be updated instead.

@JamesH65
Copy link
Contributor

JamesH65 commented Feb 27, 2017

Is the originator @maglub happy to make the changes as proposed by @bennuttall? Once done (or closed) I'll push to the comms team for checking and get it merged.

Copy link
Author

@maglub maglub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok with me

@maglub
Copy link
Author

maglub commented Feb 28, 2017

@JamesH65 you can merge this. Thanks.

@JamesH65
Copy link
Contributor

I think that Ben suggestion above needs to be implemented - make the Jessie instructions the default (can probably completely remove the mencoder stuff). Is @maglub able to do that?

@maglub
Copy link
Author

maglub commented Feb 28, 2017

To be honest, I have no idea on how to do this. I do not have the access rights to do a merge.

@JamesH65
Copy link
Contributor

JamesH65 commented Feb 28, 2017 via email

@lurch
Copy link
Contributor

lurch commented Feb 28, 2017

@maglub You can do that by making further edits to https://github.com/maglub/documentation/tree/patch-1 - they should show up here (in this PR) automatically.

@maglub
Copy link
Author

maglub commented Mar 4, 2017

=) Well, I have read through this pull request a bit better now, and realized that I am looking at my own pull request. Silly me. It is over 2 years old, so I had completely forgotten about it.

Will look into it again. Not sure it is relevant anymore.

@JamesH65
Copy link
Contributor

@maglub I think I'm going to close this - we've had some changes merged on this section over the last week, and they cover the area referenced by your PR. Thanks for your efforts.

@JamesH65 JamesH65 closed this Mar 14, 2017
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.

None yet

5 participants