Opened 2 years ago

Closed 2 years ago

#804 closed Nitpick (fixed)

Get rid of redundant -l option

Reported by: agraef Owned by: dave
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 agraef 2 years ago.

Download all attachments as: .zip

Change History (3)

Changed 2 years ago by agraef

comment:1 Changed 2 years ago by agraef

  • Component changed from Build System to Ingen

comment:2 Changed 2 years ago by dave

  • Resolution set to fixed
  • Status changed from new to closed

Good Idea. Applied in r3940, thanks.

Note: See TracTickets for help on using tickets.