Skip to content

Deleted compiled binary 'fixdep' fixes #1#2

Open
prashants wants to merge 2 commits intoiocellnetworks:masterfrom
prashants:master
Open

Deleted compiled binary 'fixdep' fixes #1#2
prashants wants to merge 2 commits intoiocellnetworks:masterfrom
prashants:master

Conversation

@prashants
Copy link
Copy Markdown
Collaborator

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

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>
Signed-off-by: Prashant Shah <pshah.mumbai@gmail.com>
@iocellnetworks
Copy link
Copy Markdown
Owner

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
Copy link
Copy Markdown
Owner

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
Copy link
Copy Markdown
Collaborator Author

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants