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

Add local-test shell script, which I use on the target #13

Open
wants to merge 1 commit into
base: dev-0.3
Choose a base branch
from

Conversation

cxw42
Copy link
Contributor

@cxw42 cxw42 commented Oct 24, 2020

Overview

Problem: I work on lots of different targets with different configurations. Not all targets have ~/.gstreamer-1.0/plugins in GST_PLUGIN_PATH.

Solution proposed by this PR: Add a shell script that:

  1. installs gst-perf in a temporary directory; and
  2. opens a shell with that directory added to GST_PLUGIN_PATH.

I have been using this approach for several months on other projects and have had good success with it.

Usage example

$ ./autogen.sh && ./configure
$ ./test-locally.sh
# ... messages as it installs into ./test-locally-install-dir
make[1]: Leaving directory '.../gst-perf'
>>>  bash
$ gst-launch-1.0 -v videotestsrc ! 'video/x-raw,framerate=30/1' ! perf ! fakesink sync=true
# ... pipeline runs, using the local perf

Other

I added an EditorConfig file that supported editors will use to apply the project's indentation settings, as I understand them.

Thank you for considering this PR!

- test-locally.sh.in: new script to install gst-perf in a temporary
  directory and open a shell with that directory in GST_PLUGIN_PATH
- configure.ac: generate test-locally.sh
- .gitignore: update to ignore the generated script and the
  temporary directory
- .editorconfig: new file that configures supported editors with the
  project's indentation settings.
@michaelgruner michaelgruner self-assigned this Oct 30, 2020
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