From: Ben Stanley <Ben.Stanley@exemail.com.au>
To: Travis Place <wishie@wishie.net>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH] alsa-info.sh - Fixed typo that stops the 'Automatic Upload' warning from appearing.
Date: Tue, 20 May 2008 23:07:21 +1000 [thread overview]
Message-ID: <1211288841.3811.1.camel@localhost> (raw)
In-Reply-To: <200805190008.30421.wishie@wishie.net>
Travis,
Thanks for fixing this. I got burned by it today running it on a
computer at work. I was none too pleased to see my machine details
uploaded.
Anyway, thanks for fixing it.
Ben Stanley.
On Mon, 2008-05-19 at 00:08 +1000, Travis Place wrote:
> Fixed a typo that was causing no warning about the script automatically
> uploading info to a pastebin. Some users were NOT too happy with this.
>
> Also, bumped up the version number so existing versions will automatically
> update to the fixed version upon next script run.
>
>
> diff -r 93548f7b2086 alsa-info.sh
> --- a/alsa-info.sh Wed Apr 30 14:47:42 2008 +0200
> +++ b/alsa-info.sh Mon May 19 00:01:04 2008 +1000
> @@ -1,6 +1,6 @@
> #!/bin/bash
>
> -SCRIPT_VERSION=0.4.43
> +SCRIPT_VERSION=0.4.44
> CHANGELOG="http://hg.alsa-project.org/alsa/log/tip/alsa-info.sh"
>
>
> #################################################################################
> @@ -215,7 +215,7 @@ if [ "$QUESTION" = "yes" ]; then
> if [ "$QUESTION" = "yes" ]; then
> if [[ -n "$DIALOG" ]]
> then
> -if [ -z "NOUPLOAD" ]; then
> +if [ -z "$NOUPLOAD" ]; then
> dialog --backtitle "$BGTITLE" --title "ALSA-Info script v
> $SCRIPT_VERSION" --yesno "\nThis script will collect information about your
> ALSA installation and sound related hardware, to help diagnose your
> problem\n\nBy default, this script will AUTOMATICALLY UPLOAD your information
> to a pastebin site.\n\nSee $0 --help for options\n\nDo you want to run this
> script?" 0 0
> else
> dialog --backtitle "$BGTITLE" --title "ALSA-Info script v
> $SCRIPT_VERSION" --yesno "\nThis script will collect information about your
> ALSA installation and sound related hardware, to help diagnose your
> problem\n\nSee $0 --help for options\n\nDo you want to run this script?" 0 0
>
>
> Thanks,
> Travis Place
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2008-05-20 13:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-28 14:10 [PATCH] alsa-info.sh - Add another distro to search for Travis Place
2008-04-30 12:45 ` Takashi Iwai
2008-05-18 14:08 ` [PATCH] alsa-info.sh - Fixed typo that stops the 'Automatic Upload' warning from appearing Travis Place
2008-05-19 16:10 ` Travis Place
2008-05-19 16:26 ` Takashi Iwai
2008-05-20 13:07 ` Ben Stanley [this message]
2008-05-20 13:44 ` Travis Place
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1211288841.3811.1.camel@localhost \
--to=ben.stanley@exemail.com.au \
--cc=alsa-devel@alsa-project.org \
--cc=wishie@wishie.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.