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

common: rpm spec improvements #2815

Merged
merged 1 commit into from
Apr 9, 2018
Merged

Conversation

krzycz
Copy link
Contributor

@krzycz krzycz commented Mar 30, 2018

Replaces "--with rpmem" with "--with fabric".
Makes "check" step to be enabled by default, unless explicilty disabled
with "_skip_check=1".
Uses default test configuration, if "_testconfig" is not supplied.
Adds version to the obsoleted "nvml-tools" subpackage.


This change is Reviewable

@krzycz krzycz force-pushed the common-rpm_spec_fixes branch from 60d78aa to f8f4048 Compare April 9, 2018 10:02
Replaces "--with rpmem" with "--with fabric".
Makes "check" step to be enabled by default, unless explicilty disabled
with "_skip_check=1".
Uses default test configuration, if "_testconfig" is not supplied.
Adds version to the obsoleted "nvml-tools" subpackage.
@krzycz krzycz force-pushed the common-rpm_spec_fixes branch from f8f4048 to 5dd6572 Compare April 9, 2018 10:32
@codecov-io
Copy link

Codecov Report

Merging #2815 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2815      +/-   ##
=========================================
+ Coverage   77.18%   77.2%   +0.01%     
=========================================
  Files         175     175              
  Lines       25176   25182       +6     
=========================================
+ Hits        19432   19441       +9     
+ Misses       5744    5741       -3

@krzycz krzycz merged commit 7ca9157 into pmem:master Apr 9, 2018
@krzycz krzycz deleted the common-rpm_spec_fixes branch April 9, 2018 12:10
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.

3 participants