~alpine/devel

1

[alpine-devel] comments on setup-apkrepos

Natanael Copa <ncopa@alpinelinux.org>
Details
Message ID
<076ca6456d9b67f0938f5f1fbd8b3720@ytre.org>
Sender timestamp
1298906499
DKIM signature
missing
Download raw message
 Hi,

 Nice work that has been done on setup-apkrepos!

 I have a few minor comments.

 1) Try avoid == operator.
 Its bash specific, and even if it is implemented in busybox ash, its no 
 guarantee it will in future.

 replace:

   if [ $var == "something" ]; then

 with:

   if [ "$var" = "something" ]; then

 For the specific case in setup-apkrepos, I think that case ... in would 
 be better:

 case "$setup_method" in
   random)
     get_random_method;;
   fastest)
     get_fastest_mirror;;
   mirror_list)
     prompt_mirror_list;;
   custom_mirror)
     prompt_clear_repositories;;
 esac


 2) I dont like the 2 y/n questions at setup-alpine end.
 Setup internet apk repositories? [y/N]?
 Change timezone (default is UTC)? [y/N]?

 I think we don't need those questions. Better would be

 How would you like to set up the apk repostories?
 1) Automatically...
 ...
 5) do not setup apk repositories at all.
 Please select how to setup APK repositories [5]:


 Then we have 1 less question to ask user.

 Same goes for timezone.

 If you are ok with it I'll change the above.

 I will have a look at the setup-disk script and then make an 
 alpine-conf 2.6 release with the new stuff.

 Thanks!


---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---
Jeff Bilyk <jbilyk@gmail.com>
Details
Message ID
<AANLkTin=fZwzna9s-Ddhu3UQYqAHucXtCyboDTiQ5FVD@mail.gmail.com>
In-Reply-To
<076ca6456d9b67f0938f5f1fbd8b3720@ytre.org> (view parent)
Sender timestamp
1298913034
DKIM signature
missing
Download raw message
On Mon, Feb 28, 2011 at 10:21 AM, Natanael Copa <ncopa@alpinelinux.org> wrote:
> Hi,
>
> Nice work that has been done on setup-apkrepos!
>
> I have a few minor comments.
>
> 1) Try avoid == operator.
> Its bash specific, and even if it is implemented in busybox ash, its no
> guarantee it will in future.
>
> replace:
>
>  if [ $var == "something" ]; then
>
> with:
>
>  if [ "$var" = "something" ]; then
>
> For the specific case in setup-apkrepos, I think that case ... in would be
> better:
>
> case "$setup_method" in
>  random)
>    get_random_method;;
>  fastest)
>    get_fastest_mirror;;
>  mirror_list)
>    prompt_mirror_list;;
>  custom_mirror)
>    prompt_clear_repositories;;
> esac
>
>
> 2) I dont like the 2 y/n questions at setup-alpine end.
> Setup internet apk repositories? [y/N]?
> Change timezone (default is UTC)? [y/N]?
>
> I think we don't need those questions. Better would be
>
> How would you like to set up the apk repostories?
> 1) Automatically...
> ...
> 5) do not setup apk repositories at all.
> Please select how to setup APK repositories [5]:
>
>
> Then we have 1 less question to ask user.
>
> Same goes for timezone.
>
> If you are ok with it I'll change the above.
I'm fine with those changes, change away :).
>
> I will have a look at the setup-disk script and then make an alpine-conf 2.6
> release with the new stuff.
>
> Thanks!
>
>
> ---
> Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
> Help:         alpine-devel+help@lists.alpinelinux.org
> ---
>
>

-- 
Jeff


---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---
Reply to thread Export thread (mbox)