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

debianpkg: Fix indentation and variable name #1558

Closed
wants to merge 1 commit into from

Conversation

rburkholder
Copy link
Contributor

@rburkholder rburkholder commented Dec 17, 2017

  • change tab to spaces to get formatting correct

Signed-off-by: Raymond Burkholder raymond@burkholder.net

@LabN-CI
Copy link
Collaborator

LabN-CI commented Dec 17, 2017

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table
_ _
Result SUCCESS git merge/1558 2ea81a9
Date 12/16/2017
Start 22:50:08
Finish 23:13:03
Run-Time 22:55
Total 1805
Pass 1805
Fail 0
Valgrind-Errors 0
Valgrind-Loss 0
Details vncregress-2017-12-16-22:50:08.txt
Log autoscript-2017-12-16-22:50:49.log.bz2

For details, please contact louberger

@rburkholder rburkholder changed the title fix indentation and variable name debianpkg: Fix indentation and variable name Dec 17, 2017
@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: FAILED

See below for issues.
CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2199/

This is a comment from an EXPERIMENTAL automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

Get source and apply patch from patchwork: Successful

Building Stage: Failed

Fedora24 amd64 build: Successful
FreeBSD11 amd64 build: Successful
NetBSD7 amd64 build: Successful
Ubuntu1604 amd64 build: Successful
NetBSD6 amd64 build: Successful
CentOS7 amd64 build: Successful
CentOS6 amd64 build: Successful
Debian8 amd64 build: Successful
FreeBSD10 amd64 build: Successful
Debian9 amd64 build: Successful
FreeBSD9 amd64 build: Successful
OmniOS amd64 build: Successful
Ubuntu1404 amd64 build: Successful
OpenBSD60 amd64 build: Successful

Ubuntu1204 amd64 build: Failed

Ubuntu1204 amd64 build: No useful log found

@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: FAILED

See below for issues.
CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2198/

This is a comment from an EXPERIMENTAL automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

Get source and apply patch from patchwork: Successful

Building Stage: Failed

Fedora24 amd64 build: Successful
Ubuntu1604 amd64 build: Successful
Ubuntu1404 amd64 build: Successful
FreeBSD11 amd64 build: Successful
Debian8 amd64 build: Successful
CentOS6 amd64 build: Successful
NetBSD6 amd64 build: Successful
CentOS7 amd64 build: Successful
Ubuntu1204 amd64 build: Successful
Debian9 amd64 build: Successful
OpenBSD60 amd64 build: Successful
NetBSD7 amd64 build: Successful
FreeBSD10 amd64 build: Successful

FreeBSD9 amd64 build: Failed

FreeBSD9 amd64 build: No useful log found

OmniOS amd64 build: Failed

OmniOS amd64 build: No useful log found

@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2194/

This is a comment from an EXPERIMENTAL automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.


CLANG Static Analyzer Summary

  • Github Pull Request 1558, comparing to Git base SHA acf43be

No Changes in Static Analysis warnings compared to base

21 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2194/artifact/shared/static_analysis/index.html

@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2200/

This is a comment from an EXPERIMENTAL automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.


CLANG Static Analyzer Summary

  • Github Pull Request 1558, comparing to Git base SHA acf43be

No Changes in Static Analysis warnings compared to base

21 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-2200/artifact/shared/static_analysis/index.html

@donaldsharp donaldsharp self-requested a review December 19, 2017 16:38
@qlyoung
Copy link
Member

qlyoung commented Dec 20, 2017

@rburkholder Please change the subject line of your git commit to read
debianpkg: Fix indentation and variable name
Additionally please sign off on your commit.

You can do this from the git cli, assuming the last commit on your branch is this one, with
$ git commit --amend -sm "debianpkg: Fix indentation and variable name"

Then update your branch here on Github with
$ git push -f <remote> <branch>

@donaldsharp
Copy link
Member

I have created #1583 that fixes the commit message @rburkholder can you take a look and make sure you are happy?

@riw777
Copy link
Member

riw777 commented Jan 2, 2018

Same as 1583, which has already been merged.

@riw777 riw777 closed this Jan 2, 2018
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.

6 participants