Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-19 Thread u
intrigeri:
 sajolida wrote (18 Jun 2015 13:32:18 GMT) :
 If there is a simple way of detecting whether the code is running from
 inside Tails or from outside Tails, [...]
 
 There is :)

So maybe, in that case, kurono can implement the new wording for from
inside Tails and either I will do the other part, or he can do that too,
which I would prefer of course.

Cheers!
u.
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-18 Thread sajolida
u:
 sajolida:
 kurono:
 Hi guys,

 Could you please review the new iteration?

 Ticket: https://labs.riseup.net/code/issues/5623
 Branch:
 https://git-tails.immerda.ch/kurono/liveusb-creator/commit/?h=feature/5623-Installer-should-refuse-empty-deviceid=ce0f3ac974a7b9f6e18071f01731e1d7edab0678

 Thanks a lot!

 Instead of:

 'The destination device was not created using Tails Installer. You can
 only upgrade a device created by Tails Installer. You need to install it
 a new to start with, e.g. by choosing the \Clone   Install\ action
 instead.'

 What about:

 'It is impossible to upgrade the destination device because it was not
 created using Tails Installer. You should instead reinstall Tails on
 this device using the \Clone  Install\ action.'

 Which is shorted and avoids saying Tails Installer twice.
 
 I am currently implementing the Install from ISO option. Maybe we can
 make this sentence future compatible somehow? :

Right :)

If there is a simple way of detecting whether the code is running from
inside Tails or from outside Tails, then we could have:

inside Tails: 'It is impossible to upgrade the destination device
because it was not created using Tails Installer. You should instead
reinstall Tails on this device using the \Clone  Install\ action.'

outside Tails: 'It is impossible to upgrade the destination device
because it was not created using Tails Installer. You should instead
reinstall Tails on this device using the \Install from ISO\ action.'

Otherwise maybe mentioning both (or none) would do:

both: 'It is impossible to upgrade the destination device because it was
not created using Tails Installer. You should instead reinstall Tails on
this device using either the \Clone  Install\ or the  \Install from
ISO\ action.'

none: 'It is impossible to upgrade the destination device because it was
not created using Tails Installer. You should instead reinstall Tails on
this device.'

I anyway think that the future is to get rid of this screen and this
error message:

https://tails.boum.org/blueprint/bootstrapping/installer#index2h2

But yes, what u mentions will arrive earlier.

-- 
sajolida
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-18 Thread intrigeri
sajolida wrote (18 Jun 2015 13:32:18 GMT) :
 If there is a simple way of detecting whether the code is running from
 inside Tails or from outside Tails, [...]

There is :)
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-17 Thread sajolida
kurono:
 Hi guys,
 
 Could you please review the new iteration?
 
 Ticket: https://labs.riseup.net/code/issues/5623
 Branch:
 https://git-tails.immerda.ch/kurono/liveusb-creator/commit/?h=feature/5623-Installer-should-refuse-empty-deviceid=ce0f3ac974a7b9f6e18071f01731e1d7edab0678
 
 Thanks a lot!

Instead of:

'The destination device was not created using Tails Installer. You can
only upgrade a device created by Tails Installer. You need to install it
a new to start with, e.g. by choosing the \Clone   Install\ action
instead.'

What about:

'It is impossible to upgrade the destination device because it was not
created using Tails Installer. You should instead reinstall Tails on
this device using the \Clone  Install\ action.'

Which is shorted and avoids saying Tails Installer twice.

-- 
sajolida
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-17 Thread u
sajolida:
 kurono:
 Hi guys,

 Could you please review the new iteration?

 Ticket: https://labs.riseup.net/code/issues/5623
 Branch:
 https://git-tails.immerda.ch/kurono/liveusb-creator/commit/?h=feature/5623-Installer-should-refuse-empty-deviceid=ce0f3ac974a7b9f6e18071f01731e1d7edab0678

 Thanks a lot!
 
 Instead of:
 
 'The destination device was not created using Tails Installer. You can
 only upgrade a device created by Tails Installer. You need to install it
 a new to start with, e.g. by choosing the \Clone   Install\ action
 instead.'
 
 What about:
 
 'It is impossible to upgrade the destination device because it was not
 created using Tails Installer. You should instead reinstall Tails on
 this device using the \Clone  Install\ action.'
 
 Which is shorted and avoids saying Tails Installer twice.

I am currently implementing the Install from ISO option. Maybe we can
make this sentence future compatible somehow? :

Cheers!
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-06-16 Thread kurono
Hi guys,

Could you please review the new iteration?

Ticket: https://labs.riseup.net/code/issues/5623
Branch:
https://git-tails.immerda.ch/kurono/liveusb-creator/commit/?h=feature/5623-Installer-should-refuse-empty-deviceid=ce0f3ac974a7b9f6e18071f01731e1d7edab0678

Thanks a lot!

El 06/05/15 a las 17:39, kurono escribió:
 Hi,
 
 Could you please review the text introduced in the liveusb-creator for
 this ticket?
 https://labs.riseup.net/code/issues/5623
 
 The modified branch is here:
 https://git-tails.immerda.ch/kurono/liveusb-creator/log/?h=feature/5623-Installer-should-refuse-empty-device
 
 Many thanks,
 
 kurono
 
 El 06/05/15 a las 12:17, intrigeri escribió:
 kurono wrote (05 May 2015 18:36:38 GMT) :
 Could you please review and merge?

 Thanks! Initial review done on the ticket.
 ___
 Tails-dev mailing list
 Tails-dev@boum.org
 https://mailman.boum.org/listinfo/tails-dev
 To unsubscribe from this list, send an empty email to 
 tails-dev-unsubscr...@boum.org.

___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-05-12 Thread kurono
Thanks!. The branch is ready for QA.

El 06/05/15 a las 19:36, sajolida escribió:
 kurono:
 Hi,

 Could you please review the text introduced in the liveusb-creator for
 this ticket?
 https://labs.riseup.net/code/issues/5623

 The modified branch is here:
 https://git-tails.immerda.ch/kurono/liveusb-creator/log/?h=feature/5623-Installer-should-refuse-empty-device
 
 The string in question is:
 
 « The selected device does not have a Tails installation, it is required
 for the upgrade process. »
 
 I'd rather say:
 
 « The destination device is not a Tails device. You can only upgrade an
 already installed Tails device. »
 
 We're in the process of rethinking the terminology around Tails devices
 and the installation process. For example we might stop saying devices
 at some point. But right now the strings in Tails Installer are quite a
 mess so let's not be too picky and stick with what's currently used in
 the code.
 
 Thanks for asking for reviews of your user strings. Don't hesitate to do
 that more often.
 
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-05-06 Thread intrigeri
kurono wrote (05 May 2015 18:36:38 GMT) :
 Could you please review and merge?

Thanks! Initial review done on the ticket.
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-05-06 Thread sajolida
kurono:
 Hi,
 
 Could you please review the text introduced in the liveusb-creator for
 this ticket?
 https://labs.riseup.net/code/issues/5623
 
 The modified branch is here:
 https://git-tails.immerda.ch/kurono/liveusb-creator/log/?h=feature/5623-Installer-should-refuse-empty-device

The string in question is:

« The selected device does not have a Tails installation, it is required
for the upgrade process. »

I'd rather say:

« The destination device is not a Tails device. You can only upgrade an
already installed Tails device. »

We're in the process of rethinking the terminology around Tails devices
and the installation process. For example we might stop saying devices
at some point. But right now the strings in Tails Installer are quite a
mess so let's not be too picky and stick with what's currently used in
the code.

Thanks for asking for reviews of your user strings. Don't hesitate to do
that more often.

-- 
sajolida
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


Re: [Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-05-06 Thread kurono
Hi,

Could you please review the text introduced in the liveusb-creator for
this ticket?
https://labs.riseup.net/code/issues/5623

The modified branch is here:
https://git-tails.immerda.ch/kurono/liveusb-creator/log/?h=feature/5623-Installer-should-refuse-empty-device

Many thanks,

kurono

El 06/05/15 a las 12:17, intrigeri escribió:
 kurono wrote (05 May 2015 18:36:38 GMT) :
 Could you please review and merge?
 
 Thanks! Initial review done on the ticket.
 ___
 Tails-dev mailing list
 Tails-dev@boum.org
 https://mailman.boum.org/listinfo/tails-dev
 To unsubscribe from this list, send an empty email to 
 tails-dev-unsubscr...@boum.org.
 
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.


[Tails-dev] [review'n'merge:1.4.1] feature/5623-Installer-should-refuse-empty-device

2015-05-05 Thread kurono
Hi,

Ticket:
https://labs.riseup.net/code/issues/5623

Feature branch:
https://git-tails.immerda.ch/kurono/liveusb-creator/log/?h=feature/5623-Installer-should-refuse-empty-device

Could you please review and merge?

Thanks,

kurono
___
Tails-dev mailing list
Tails-dev@boum.org
https://mailman.boum.org/listinfo/tails-dev
To unsubscribe from this list, send an empty email to 
tails-dev-unsubscr...@boum.org.