Hacker News Reader

Comment by FreeFull | Hacker News Reader

I'm having trouble compiling this.

    freefull@freefull-hp ~/c/o/devos> make all
    Building Kernel
    make -C ./kernel
    make[1]: Entering directory `/home/freefull/code/other/devos/kernel'
    makefile:9: runtime/Makefile: No such file or directory
    makefile:10: core/Makefile: No such file or directory
    makefile:11: modules/Makefile: No such file or directory
    makefile:12: arch/x86/Makefile: No such file or directory
    make[1]: *** No rule to make target `arch/x86/Makefile'.  Stop.
    make[1]: Leaving directory `/home/freefull/code/other/devos/kernel'
    make: *** [all] Error 2

View on HN

I haven't tried myself but it looks like a casing issue. All the makefiles have a lower case m. I'd suspect correcting that would do that trick.

make looks for both Makefile and makefile, but those directories don't have any makefiles at all.

Is there a configure script or cmake or something? It looks like you might have to do something before `make all`

Nope, just a top level makefile and a readme that says to compile you only need to

    make all

make looks for [Mm]akefile if you run it with no arguments, but here it's being run recursively (or included). On OSX, filenames are case-insensitive, which could be a possible reason why it works for some people. On Linux, filenames are case sensitive.

That seems to be the case, since running file on one of the existing .o files says that they are Mach-O i386 objects. The readme doesn't mention OS X though, only Ubuntu

Well, on my system (Ubuntu) at least this is not the case and renaming the files gets past the errors and on to actual build errors:

  runtime/cxx.cc:74:41: error: ‘operator new’ takes type ‘size_t’ (‘unsigned int’) as first parameter [-fpermissive]
  runtime/cxx.cc:84:43: error: ‘operator new’ takes type ‘size_t’ (‘unsigned int’) as first parameter [-fpermissive]
  make[1]: *** [runtime/cxx.o] Error 1

After renaming the Makefiles to fix the case, running make clean and then make all, I get the same error.

Thinking about this, this is a casing issue after all. The makefiles include Makefile rather than makefile

The compile/build works fine now (check out my commits).

However what does not yet work is booting the image with Qemu.

Thanks for fixing the compile

Reply on news.ycombinator.com