iocellnetworks / ndas4linux

The Linux driver and device management program for NDAS devices such as the NetDISK.
GNU General Public License v2.0
13 stars 12 forks source link

Deleted compiled binary 'fixdep' fixes #1 #2

Open prashants opened 11 years ago

prashants commented 11 years ago

The binary was compiled on 32 bit arch due to which 'make' failed on 64 bit arch's. Deleting 'fixdep' will cause it to be recompiled on the next 'make'

Signed-off-by: Prashant Shah pshah.mumbai@gmail.com

iocellnetworks commented 11 years ago

It is correct. The compiled fixdep should not be uploaded into the source tree.

After making and testing, the clean command should remove it also.

example

make linux64-dev clean
or make linux-dev clean

These should take away the compiled version before commiting any other changes.

iocellnetworks commented 11 years ago

I remember that those type compiler errors came up a bunch when I compiled the 64 bit version, but did not seem to affect usability. I later on began to leave them alone, since they would happen again on the 32Bit version. It was a tricky one for me, that I did not fully discover the right solution for.

Most important I think is that those are only debug messages. So I think they did not affect usability. I've run the -rel version instead on Fedora, Debian, Mint and Ubuntu since then, because the log files got quite big when I used -dev.

prashants commented 11 years ago

The clean option is broken on ubuntu 64 bit. May be you can add the binaries to the gitignore file.

I am testing this from a normal user perspective :)