Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update readme.txt #117

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update readme.txt #117

wants to merge 3 commits into from

Conversation

Rossolson
Copy link

Here is a draft update for showing the full Installation steps. I hope this is helpful, but feel free to ignore it. : )

Actual Installation steps documented.
readme.txt Outdated
There are three ways to install, depending on your usage.
* Download the ANSICON zip file (including the executables) from the
Releases page (https://github.com/adoxa/ansicon/releases) or
http://adoxa.altervista.org/ansicon/ or http://ansicon.adoxa.vze.com/

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume people reading the readme have already downloaded it; even if they're reading it online, the links are available elsewhere. In any event, vze is the url to use, in case something happens with altervista (but let's hope not).

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adoxa can only speak from my experience, but not knowing GitHub it took me some time to figure out where to get the executables from.

readme.txt Outdated
"System32" on a 64-bit system). This means you explicitly run 'ansicon'
whenever you want to use it.
* Unzip the files into a directory such as C:\Program Files (x86)\ansicon\
(for 32-bit OS) or into C:\Program Files\ansicon\ (for 64-bit OS).

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really don't like this, as it requires being administrator, which you haven't mentioned. Besides which, (x86) only exists on 64-bit systems, so a 32-bit system should still be installing to Program Files.

I've always assumed that anyone using ANSICON would already be familiar with the command line and would be familiar with how to install it. Given the issues that have come up, that would appear to be a false assumption.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you're seeing the result of two things: 1) People having less and less understanding of the basics of the Command Prompt, so the READMEs have to have true step-by-step instructions. 2) The ongoing efforts of the Copy-and-Paste-Examples group to make it nearly brainless to get open source software up and running. This is my attempt to help move that forward on such a foundational piece of software like ANSICON.

Simplification for 32/64-bit differences.
Additional 32/64 bit differences eliminated.
@Rossolson
Copy link
Author

I've updated the README to fix my assumptions about the 32-bit environment's file paths for Program Files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants