Move and modularize the install scripts

face-hh / webx

An alternative for the World Wide Web - browse websites such as buss://yippie.rizz made in HTML, CSS and Lua. Custom web browser, custom HTML rendering engine, custom search engine, and more.

Apache License 2.0

1.67k stars 155 forks source link

Move and modularize the install scripts #137

Closed QuinceTart10 closed 1 month ago

QuinceTart10 commented 1 month ago

I think everything is clear from the commits + idk how to describe this

QuinceTart10 commented 1 month ago

@Helloyunho could you test out the modified macOS script please? I need to know that it still works.

Helloyunho commented 1 month ago

@Helloyunho could you test out the modified macOS script please? I need to know that it still works.

Sure, give me a minute

Helloyunho commented 1 month ago

@QuinceTart10 It works fine, but I think I forgot to first remove the old Napture installation and then move the new one

# For example
rm -rf /Applications/Napture.app || true # ignore fails
mv target/release/Napture.app /Applications || exit 1

Can you do that for me? (or if there's a flag in mv that does this automatically you can use that as well)

Helloyunho commented 1 month ago

@QuinceTart10 you missed || true part which is important since it will raise error when the app is not already installed

QuinceTart10 commented 1 month ago

@QuinceTart10 you missed || true part which is important since it will raise error when the app is not already installed

Ah, sorry. Let me do it real quick.

ByzantineProcess commented 1 month ago

works!