Skip to content
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.

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

Change GNUmakefile to generate a shared library #35

Closed
wants to merge 4 commits into from

Conversation

Sufrostico
Copy link

Add GNUmakefile targets to generate, install and purge a libstemmer.so shared library intended to be dynamically linked.

This should facilitate the implementation of wrapper libraries and the integration of this software with other applications.

Pending:

  • Implement version numbering for the .so file

This is in response to issue #34

In brief, the term position independent code (PIC) refers to the generated
machine code which is memory address agnostic, i.e. does not make any
assumptions about where it was loaded into RAM. Only position independent code
is supposed to be included into shared objects (SO) as they should have an
ability to dynamically change their location in RAM.
Add make target to the GNUmakefile to install the shared library for system wide
usage in to the following folders:

    inlude/libstemmer.h  --> /usr/include/libstemmer.h
    libstemmer.so --> /usr/lib/libstemmer.so

This allow to compile applications using libstemmer with the command
    gcc -lstemmer file.c

file.c should use the include in the following format:
    #include <libstemmer.h>
@ojwb
Copy link
Member

ojwb commented Mar 30, 2016

@Sufrostico in #34 you say mitya57's patch is better than yours - do you want to abandon this PR?

@Sufrostico
Copy link
Author

Yes, I'll take a look into @mitya57 patch and try to merge it with snowball last version and PR again.

Thanks for the attention to this PR.

@ojwb
Copy link
Member

ojwb commented Mar 31, 2016

OK, closing then.

@ojwb ojwb closed this Mar 31, 2016
@Sufrostico Sufrostico deleted the shared_library branch June 30, 2016 15:09
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