Strange wording in en-Linux-Install-Script.md about jamuluswebsite HOT 8 CLOSED

In fact, this would also be something for the new "blog" or community tricks page.

Agreed: #97 (comment) (BTW I removed the link to it from the install page until such time as we can verify that the content is in fact correct - so expose it to the community first in other words)

from jamuluswebsite.

There is also typo, missing " here:

* `DISTRO="Ubuntu` for a Ubuntu or Linux Mint
* `DISTRO="Debian` for a Debian or Raspian Linux
* `DISTRO="Fedora` for a Fedora Linux

Should be

* `DISTRO="Ubuntu"` for a Ubuntu or Linux Mint
* `DISTRO="Debian"` for a Debian or Raspian Linux
* `DISTRO="Fedora"` for a Fedora Linux

I'll do the fix in the French version.

I can submit the fix for the EN version too, in this case, in wich branch? changes?

from jamuluswebsite.

And just after, for :

Furthermore if the installation is dependent of the release the variable `LINVERSION` is introduced but is currently not used. In the Ubuntu if statement there is an example how version dependent installation calls can be performed.

It's ambiguous, I suggest to enclose the statement if in "`":

Furthermore if the installation is dependent of the release the variable `LINVERSION` is introduced but is currently not used. In the Ubuntu `if` statement there is an example how version dependent installation calls can be performed.

from jamuluswebsite.

I don't know who is in charge of the install script, but it would be great if he changed it.

In fact, this would also be something for the new "blog" or community tricks page.

from jamuluswebsite.

I'm only reading this thread after I've reviewed the French translation of this page.
Doing so, I've saw a few typo in the English version and I already made #97 for that.
This PR contains a fix for the first issue you're mentionning here @gegeweb . Please check if that's fine for you.

Now, I've added the missing " character you're mentionning in your second message here. Good eye!

I agree here too with your third point (the if) and am adding it right now to the PR too.

from jamuluswebsite.

#171 is the new pr

from jamuluswebsite.

@ann0see closing here then?

from jamuluswebsite.

Yes closed

from jamuluswebsite.