polard8 / kernel

Polar D8 kernel.
https://github.com/polard8/kernel
MIT License
646 stars 35 forks source link

Running the Makefile Requires Superuser #21

Open ghost opened 5 years ago

ghost commented 5 years ago

Call me paranoid, but I'm very weary to run someones build scripts as a superuser.

It would be nice if visitors of the project could compile without feeling like their putting their system at risk.

robertapengelly commented 4 years ago

Call me paranoid, but I'm very weary to run someones build scripts as a superuser.

It would be nice if visitors of the project could compile without feeling like their putting their system at risk.

Superuser is required when using mount. The project will build successfully without superuser but you cannot create the image without requiring superuser.

ghost commented 4 years ago

@robertapengelly I'm aware of that. The primary options are either using a FUSE driver or rolling your own utility for creating the image. Neither of those are very hard to do since there's a lot of FAT code on the internet.

And then there's always the option of building the image in a VM with Travis CI, and uploading the image to GitHub releases. That would fix the problem too because then nobody would have to run the build system to try the OS.

robertapengelly commented 4 years ago

@tay10r I understand that there's the option of building it and uploading but maybe he hasn't figured that out yet. Also looking at other things on here I don't know if he's able to understand these messages cause everything else is in a different language so it may be worth someone forking the project and offering support that way.

ghost commented 4 years ago

@robertapengelly Maybe you're right that it'd be more helpful to submit a PR or something. I'll close this and try to give it a shot at some point.

frednora commented 1 year ago

nice if visitors of the project could compile without feeling like their putting their system at risk.

You are very right. But i really don't know how to fix it. I need some help to change the Makefile to do the things in the right way.