bargees / barge-os

Yet another lightweight Linux distribution for Docker containers
GNU General Public License v2.0
163 stars 15 forks source link

Adds a run.sh for quick testing under AEMU #84

Closed prologic closed 6 years ago

prologic commented 6 years ago

Also adds a screenshot of this working.

ailispaw commented 6 years ago

script directory goes to the container to build barge. So contrib/qemu/ might be a good place for it to be and also we need to put the contrib into .dockerignore. Could you move the screenshot to the directory and make REAME.md accordingly there?

ailispaw commented 6 years ago

And FYI: https://github.com/bargees/barge-packer/tree/master/qemu

prologic commented 6 years ago

Done

prologic commented 6 years ago

And FYI: https://github.com/bargees/barge-packer/tree/master/qemu

Hmm interesting didn't see this and not familiar with Packer

ailispaw commented 6 years ago

I meant that the screenshot in REAME.md and the image itself should be the same directory contrib/qemu/. And cold you make another REAME.md there for qemu and the screenshot instead of the main README.md?

ailispaw commented 6 years ago

You may need virtual HDD and port-forwarding like this.
https://github.com/bargees/barge-os/issues/75#issuecomment-381807749

prologic commented 6 years ago

Removed the screenshot; it doesn't add much value if it'll be in a separate README.

prologic commented 6 years ago

You may need virtual HDD and port-forwarding like this.

Not necessary IHMO as this is only for quick boot-up of the built images and testing. I don't see this as real production usage at all.

ailispaw commented 6 years ago

Oh now I understand that it's for testing but an example.

prologic commented 6 years ago

Pretty much ;) I borrowed it from vallinux's scripts :)

ailispaw commented 6 years ago

Removed the screenshot; it doesn't add much value if it'll be in a separate README.

I don't think so, even it's just a test's result. I like it.

prologic commented 6 years ago

Fine but I won't put it in a README :)

ailispaw commented 6 years ago

Do you mean the main README, a separate one, or both?

prologic commented 6 years ago

Do you mean the main README, a separate one, or both?

The main README is already "too full" and no-one will read another README tucked away in another corner :) -- For the really curious they can "go look" :)

ailispaw commented 6 years ago

OK. I will merge run.sh only for now. Thank you. :)

prologic commented 6 years ago

We'l see :) "iterate and move fast" :)

ailispaw commented 6 years ago

Hmm. It's already merged. Should be closed.