Opened 16 years ago
Closed 11 years ago
#3763 closed enhancement (fixed)
libroot + some other stuff for ARM
Reported by: | pfoetchen_ | Owned by: | axeld |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | System/libroot.so | Version: | R1/pre-alpha1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | arm |
Description
I got libroot to compile and some function should already work but this code is still far from usable and still contains a lot of unchanged m68k code ;)
Attachments (2)
Change History (8)
by , 16 years ago
Attachment: | libroot.diff added |
---|
by , 16 years ago
Attachment: | headers.diff added |
---|
comment:1 by , 16 years ago
Component: | - General → System/libroot.so |
---|---|
Type: | bug → enhancement |
comment:2 by , 16 years ago
I just had a quick look at the diffs. There's lot of code copied from other ports -- which is OK -- but with a lot of unmodified left-overs from m86k and PPC. Please run through the patch again and adjust header guards, add your copyright where you've changed more than trivial stuff (and extend "Haiku Inc." copyrights to 2009), and remove stuff that doesn't apply. I'd rather see empty stubs with TODOs than copied code that doesn't apply (even if the m68k port does that, too).
Anyway, nice to see you make progress. Besides getting libroot compiling, I wouldn't concentrate too much on it yet, though. Before it becomes relevant at all (save for the parts that are used in the kernel (src/system/kernel/lib)) you have to get the kernel boot to the very end, which is the main chunk of the work.
comment:3 by , 15 years ago
Platform: | All → arm |
---|
comment:5 by , 11 years ago
Yes the ticket can be closed. This is all included in the repository now.
comment:6 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
added/modified headers