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

Formatting: JsThemis native extension #412

Merged
merged 9 commits into from Mar 7, 2019

Commits on Mar 5, 2019

  1. Formatting: JsThemis native extension

    Add native extension of JsThemis to "make fmt" target group.
    
    This requires quite a bit of effort because clang-tidy insists on having
    access to *all* source code involved in complation, including all those
    NodeJS headers and its modules.
    
    First of all, we have to add some hardcoded paths to NodeJS headers.
    I have not found a proper way to get these paths so we just hardcode
    them and hope that this is fine.
    
    Then we need to have <nan.h> header file for which we have to install
    the nan module. We do a little hack to achieve that, pretending to
    'format' the installation path for Node modules.
    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    27b92bd View commit details
    Browse the repository at this point in the history
  2. Optimize, group, and reorder includes in JsThemis

    This will make automated formatting a bit more nice.
    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    2639ee8 View commit details
    Browse the repository at this point in the history
  3. Reformat JsThemis sources

    forelocked-beobachter authored and ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    02fade0 View commit details
    Browse the repository at this point in the history
  4. Add missing empty lines

    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    b1fe490 View commit details
    Browse the repository at this point in the history
  5. Add NodeJS to format checking image

    Install npm and nodejs as we need them for checking JsThemis native
    source code. At least for now.
    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    1eb8658 View commit details
    Browse the repository at this point in the history
  6. Add a warning if NodeJS is not installed

    Print out a warning when formatting files if NodeJS is not installed and
    we are not able to reformat JsThemis files.
    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    4c01a70 View commit details
    Browse the repository at this point in the history
  7. Add symlink to jsthemis.mk for Rust's libthemis-src

    The current build system for libthemis-src is quite messed up and
    requires peculiar symlinking to work. Add a symlink for the new file
    included from the main Makefile.
    ilammy committed Mar 5, 2019
    Configuration menu
    Copy the full SHA
    037cb34 View commit details
    Browse the repository at this point in the history

Commits on Mar 7, 2019

  1. Configuration menu
    Copy the full SHA
    3233ec3 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    61599f2 View commit details
    Browse the repository at this point in the history