Opened 6 years ago

Closed 6 years ago

#804 closed Nitpick (fixed)

Get rid of redundant -l option

Reported by: Albert Graef Owned by: David Robillard
Priority: minor Component: Ingen
Keywords: Cc:

Description

ingen currently requires the -l to load a patch on startup. This seems redundant when the patch file or directory could be just specified as an ordinary argument (which currently don't seem to be used, even though the Raul command line configuration records them).

The -l option also gets in the way if you want to turn ingen into an icon or dock item which is supposed to just run the program when clicked, or open an existing patch when it is dragged and dropped on the icon. The current way just makes this kinds of stuff unnecessarily complicated.

Suggested patch attached. This keeps the existing -l option intact so that it can still be used. NOTE: This also includes a minor modification in Raul to make the contents of the _files variable available.

Attachments (1)

ingen-main.patch (1.5 KB) - added by Albert Graef 6 years ago.

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by Albert Graef

Attachment: ingen-main.patch added

comment:1 Changed 6 years ago by Albert Graef

Component: Build SystemIngen

comment:2 Changed 6 years ago by David Robillard

Resolution: fixed
Status: newclosed

Good Idea. Applied in r3940, thanks.

Note: See TracTickets for help on using tickets.