All of lore.kernel.org
 help / color / mirror / Atom feed
* [alsa-devel] alsa-info: lots of mostly syntactic changes
       [not found] <1571851836428162098-webhooks-bot@alsa-project.org>
@ 2019-10-23 17:30 ` GitHub pull_request - opened
  0 siblings, 0 replies; 2+ messages in thread
From: GitHub pull_request - opened @ 2019-10-23 17:30 UTC (permalink / raw)
  To: alsa-devel

alsa-project/alsa-utils pull request #29 was opened from E5ten:

- remove all bashisms and change shebang to #!/bin/sh
- replace use of which with command -v for portability
- remove awk dependency
- general cosmetic consistency:
    - indentation
    - always put then or done on the same line as if, for, or while
- quote much more, especially when the variable refers to a file
- replace many uses of >> $FILE with command grouping and single
  redirects of the command groups
- simplify text processing pipelines, mostly by turning them into
  a single sed command

Request URL   : https://github.com/alsa-project/alsa-utils/pull/29
Patch URL     : https://github.com/alsa-project/alsa-utils/pull/29.patch
Repository URL: https://github.com/alsa-project/alsa-utils
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [alsa-devel] alsa-info: lots of mostly syntactic changes
       [not found] <1576533357364182949-webhooks-bot@alsa-project.org>
@ 2019-12-16 21:56 ` GitHub pull_request - edited
  0 siblings, 0 replies; 2+ messages in thread
From: GitHub pull_request - edited @ 2019-12-16 21:56 UTC (permalink / raw)
  To: alsa-devel

alsa-project/alsa-utils pull request #29 was edited from E5ten:

- remove all bashisms and change shebang to #!/bin/sh
- replace use of which with command -v for portability
- remove awk dependency
- general cosmetic consistency:
    - indentation
    - always put then or do on the same line as if, for, or while
- quote much more, especially when the variable refers to a file
- replace many uses of >> $FILE with command grouping and single
  redirects of the command groups
- simplify text processing pipelines, mostly by turning them into
  a single sed command

Request URL   : https://github.com/alsa-project/alsa-utils/pull/29
Patch URL     : https://github.com/alsa-project/alsa-utils/pull/29.patch
Repository URL: https://github.com/alsa-project/alsa-utils
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-12-16 21:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1571851836428162098-webhooks-bot@alsa-project.org>
2019-10-23 17:30 ` [alsa-devel] alsa-info: lots of mostly syntactic changes GitHub pull_request - opened
     [not found] <1576533357364182949-webhooks-bot@alsa-project.org>
2019-12-16 21:56 ` GitHub pull_request - edited

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.