levylabpitt / AFM-Lithography

BSD 3-Clause "New" or "Revised" License
7 stars 0 forks source link

Update install location #45

Closed ciozi137 closed 3 years ago

ciozi137 commented 4 years ago

Should be C:\Program Files (x86)\LevyLab\AFM Lithography\ and Start Menu\LevyLab\AFM like this: image

not this: image

ciozi137 commented 4 years ago

also: image

ciozi137 commented 4 years ago

@Monkeymerlot I just updated the build spec

Monkeymerlot commented 3 years ago

@ciozi137 Has this been finished with the new build script?

ciozi137 commented 3 years ago

The location is specified in the project build specs in the project. The build script only produces the self-extracting installer. If we believe my comment from July then I think it has been set

Monkeymerlot commented 3 years ago

It looks like it was correct: image

ciozi137 commented 3 years ago

That looks good. How about the Start Menu shortcut?

Monkeymerlot commented 3 years ago

@ciozi137 The shortcut menu doesn't seem to be correct. I reopened this issue until this is fixed... image image

ciozi137 commented 3 years ago

I'll look at it tomorrow. It should be simple to fix. It can be tricky to get the shortcuts right

ciozi137 commented 3 years ago

@Monkeymerlot just updated develop branch. There was also a merge commit, I tried to take your changes but just double check.

Monkeymerlot commented 3 years ago

It seems like it merged smoothly. I wasn't working on that file so we should be good.

Monkeymerlot commented 3 years ago

image It looks good now...