Page MenuHomeSolus

Helm Software Synthesizer
AbandonedPublic

Authored by css459 on Jan 14 2019, 2:39 PM.

Details

Reviewers
DataDrake
Group Reviewers
Triage Team
Summary

This commit contains the files of Helm, a software synthesizer.
This project is licensed under GPL 3, as is the JUCE library
dependency for this project.

I am not the creator of this project, but I am helping to
release it for Solus since I enjoyed using it. More information
about the project can be found at the project's home page:

https://tytel.org/helm/

The source can be found at:

https://github.com/mtytel/helm

CHANGES IN THIS COMMIT

Added the package.yml et al. as required by Solus.
The Makefile has been changed to run that which
resides in the parent directory.

TESTING

Building this project results in a standalone program, and
modules for use in DAW programs. Upon running the binary,
the user should be presented with the control cluster.

I have not written formal unit testing for this program,
but I have at least confirmed the functionality of the
program by installing it to my system and have ensured
that all files (per the site's FAQ) are present and in
the proper location:

GNU/Linux:

Standalone - /usr/bin/helm

LV2 - /usr/lib/lv2/helm.lv2

VST - /usr/lib/lxvst/helm.so (if installed)

Local Presets - ~/.helm/patches

Factory Files - /usr/share/helm/

Config File - ~/.helm/Helm.config

I have also built the project using the traditional
build procedure (make, make install) and can confirm
that the behavior is identical.

REMARKS

More information about the JUCE license and package request
can be found at the following Solus Dev post:

https://dev.getsol.us/T6358

Signed-off-by: Cole Smith <cole.s.smith1@gmail.com>

Test Plan
  • install helm
  • run GUI application
  • confirm sound output on speakers
  • confirm changing oscillators affects sound
  • confirm files exist:
    • Standalone - /usr/bin/helm
    • LV2 - /usr/lib/lv2/helm.lv2
    • VST - /usr/lib/lxvst/helm.so (if installed)
    • Local Presets - ~/.helm/patches
    • Factory Files - /usr/share/helm/
    • Config File - ~/.helm/Helm.config

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

css459 created this revision.Jan 14 2019, 2:39 PM
css459 requested review of this revision.Jan 14 2019, 2:39 PM
css459 retitled this revision from SUMMARY to Helm Software Synthesizer.Jan 14 2019, 3:36 PM
Jacalz added a subscriber: Jacalz.Jan 14 2019, 4:40 PM
Jacalz added inline comments.
package.yml
28

Build reps should be before the run dependencies and they should be sorted alphabetically

Jacalz added inline comments.Jan 14 2019, 4:44 PM
package.yml
5

I think you could remove the comments

css459 updated this revision to Diff 12354.Jan 14 2019, 5:03 PM

Fixed sort order of deps and alphabetical sort of pkg-config deps

DataDrake requested changes to this revision.Jan 16 2019, 12:53 PM
DataDrake added a subscriber: DataDrake.
DataDrake added inline comments.
package.yml
6

There is no reason to use a git source when a tarball is provided and they aren't using submodules. Additionally, you would have then gone to:

https://github.com/mtytel/helm/releases

and seen that this is a pre-release which means I cannot accept this patch.

10

This should be one continuous line without newlines.

26

You don't need to list to rundeps for things that are automatically detected by ypkg. None of these are needed.

This revision now requires changes to proceed.Jan 16 2019, 12:53 PM
css459 updated this revision to Diff 12385.EditedJan 16 2019, 3:10 PM

Change Summary

  • Changed source to tarball
  • Fixed Desc. formatting
  • Removed unneeded rundeps

I understand this is still pre-release software, so I reached out to the developer to see if an official release is coming soon. The software appears to gave good traction in the music community and about 1k stars on GitHub. Of course, it is ultimately your decision if that traction is indeed significant, so I submitted the changes requested. Thanks again for your patience as this is my first Solus package I have worked to release.

Any news on this?

JoshStrobl abandoned this revision.Oct 1 2019, 12:38 PM