8000 Regression: Failing to build static libraries · Issue #92 · szaghi/FoBiS · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Regression: Failing to build static libraries #92

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
cmacmackin opened this issue Dec 23, 2016 · 9 comments
Closed

Regression: Failing to build static libraries #92

cmacmackin opened this issue Dec 23, 2016 · 9 comments
Assignees
@szaghi

Comments

@cmacmackin
Copy link
Contributor

Thanks again for including dependency libraries in the link command to produce shared libraries. However, this has introduced a regression when building static libraries. You are trying to include the libraries in the ar command, which does not accept them. as such, you get error messages like this:

Executing: ar -rcs build/libflogging.a build/obj/logging.o  -lface  -L/home/chris/.local/lib  ; ranlib build/libflogging.a
ar: invalid option -- 'e'
Usage: ar [emulation options] [-]{dmpqrstx}[abcDfilMNoPsSTuvV] [--plugin <name>] [member-name] [count] archive-file file...
       ar -M [<mri-script]
 commands:
  d            - delete file(s) from the archive
  m[ab]        - move file(s) in the archive
  p            - print file(s) found in the archive
  q[f]         - quick append file(s) to the archive
  r[ab][f][u]  - replace existing or insert new file(s) into the archive
  s            - act as ranlib
  t            - display contents of archive
  x[o]         - extract file(s) from the archive
 command specific modifiers:
  [a]          - put file(s) after [member-name]
  [b]          - put file(s) before [member-name] (same as [i])
  [D]          - use zero for timestamps and uids/gids (default)
  [U]          - use actual timestamps and uids/gids
  [N]          - use instance [count] of name
  [f]          - truncate inserted file names
  [P]          - use full path names when matching
  [o]          - preserve original dates
  [u]          - only replace files that are newer than current archive contents
 generic modifiers:
  [c]          - do not warn if the library had to be created
  [s]          - create an archive index (cf. ranlib)
  [S]          - do not build a symbol table
  [T]          - make a thin archive
  [v]          - be verbose
  [V]          - display the version number
  @<file>      - read options from <file>
  --target=BFDNAME - specify the target object format as BFDNAME
 optional:
  --plugin <p> - load the specified plug-in
 emulation options: 
  No emulation specific options
ar: supported targets: elf64-x86-64 elf32-i386 elf32-iamcu elf32-x86-64 a.out-i386-linux pei-i386 pei-x86-64 elf64-l1om elf64-k1om elf64-little elf64-big elf32-little elf32-big pe-x86-64 pe-bigobj-x86-64 pe-i386 plugin srec symbolsrec verilog tekhex binary ihex
ranlib: 'build/libflogging.a': No such file
@szaghi szaghi self-assigned this Dec 24, 2016
@szaghi szaghi added the bug label Dec 24, 2016
@szaghi
Copy link
Owner
szaghi commented Dec 24, 2016

@cmacmackin

Chris, I was almost sure that the new get links function I have introduced was not bullet-proof (with Travis I have some difficulties to add interdependent tests). Thank you very much for reporting it, I'll fix soon.

Cheers

szaghi added a commit that referenced this issue Dec 24, 2016
@szaghi
bugfix #92
050d1fd
@szaghi
Copy link
Owner
szaghi commented Dec 24, 2016

@cmacmackin

I have just uploaded a patched-master branch, I'll push on PyPi when you confirm it works.

szaghi added a commit that referenced this issue Dec 24, 2016
@szaghi
bugfix #92
0b6c3e9
@cmacmackin
Copy link
Contributor Author

Thanks @szaghi, it works now!

@szaghi
Copy link
Owner
szaghi commented Dec 28, 2016

Done in v2.2.1, also uploaded on PyPi.

@szaghi szaghi closed this as completed Dec 28, 2016
@szaghi szaghi removed the bug label Dec 28, 2016
@cmacmackin
Copy link
Contributor Author

These fixes don't seem to have made it into the release directory. As such, when I install from PyPI they are not present. For some reason, when I was testing your pre-release commits, the release directory was updated and everything worked fine. I must have run some Pybuilder command at some point, I guess.

@szaghi
Copy link
Owner
szaghi commented Dec 29, 2016

@cmacmackin Chris, thank for reporting! Yesterday when I created the PyPi release I should had made some errors, there was a lot of noise in my office.... I'll try to correct today.

@szaghi szaghi reopened this Dec 29, 2016
@szaghi
Copy link
Owner
szaghi commented Dec 29, 2016

@cmacmackin

Chris, the new v2.2.2 on PyPi should be now up-to-date with this master. Sorry for the messy!

@szaghi szaghi closed this as completed Dec 29, 2016
@giacrossi
Copy link
Contributor
giacrossi commented Dec 29, 2016 via email

@szaghi
Copy link
Owner
szaghi commented Dec 29, 2016

@cmacmackin @giacombum

Dear Giacomo, you are wrong (as often happen to young men): there are only 2 kind of bikers, who had fallen and who are not yet fallen, but will do soon 😄

I had fallen, but I am still running...

szaghi added a commit that referenced this issue Mar 21, 2017
@szaghi
Merge tag 'v2.2.1' into develop
205451f 
Bugfix #92, static library issue

Bugfix #92, static library issue, failing regression tests.

Stable release, fully backward compatible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@szaghi szaghi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@szaghi @giacrossi @cmacmackin
< 2AC8 /div>
0