Skip to content
This repository was archived by the owner on Nov 15, 2022. It is now read-only.

Fix #22234: Update libpam4j from 1.3 to 1.8#22229

Merged
yaminikb merged 6 commits intojavaee:masterfrom
mskdeepak-oracle:libpam
Aug 22, 2017
Merged

Fix #22234: Update libpam4j from 1.3 to 1.8#22229
yaminikb merged 6 commits intojavaee:masterfrom
mskdeepak-oracle:libpam

Conversation

@mskdeepak-oracle
Copy link
Copy Markdown
Member

@mskdeepak-oracle mskdeepak-oracle commented Aug 21, 2017

Fix #22234: Libpam4j update

@mskdeepak-oracle
Copy link
Copy Markdown
Member Author

@glassfishrobot Run CI Tests please

@glassfishrobot
Copy link
Copy Markdown
Contributor

Starting CI tests run

@mskdeepak-oracle mskdeepak-oracle self-assigned this Aug 21, 2017
@glassfishrobot
Copy link
Copy Markdown
Contributor

All CI tests successful

@mskdeepak-oracle mskdeepak-oracle changed the title NoBug: Update libpam4j from 1.3 to 1.8 Fix #22234: Update libpam4j from 1.3 to 1.8 Aug 22, 2017
@mskdeepak-oracle
Copy link
Copy Markdown
Member Author

@glassfishrobot Run CI Tests please

@glassfishrobot
Copy link
Copy Markdown
Contributor

Starting CI tests run

@glassfishrobot
Copy link
Copy Markdown
Contributor

All CI tests successful

@yaminikb
Copy link
Copy Markdown
Member

@glassfishrobot run ci tests please

@glassfishrobot
Copy link
Copy Markdown
Contributor

Starting CI tests run

yaminikb
yaminikb previously approved these changes Aug 22, 2017
Copy link
Copy Markdown
Member

@yaminikb yaminikb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@glassfishrobot
Copy link
Copy Markdown
Contributor

All CI tests successful

@mskdeepak-oracle
Copy link
Copy Markdown
Member Author

@glassfishrobot run ci tests please

@glassfishrobot
Copy link
Copy Markdown
Contributor

Starting CI tests run

@glassfishrobot
Copy link
Copy Markdown
Contributor

All CI tests successful

@yaminikb yaminikb merged commit cf5d56c into javaee:master Aug 22, 2017
<version>${project.version}</version>
<groupId>org.glassfish.external</groupId>
<artifactId>libpam4j</artifactId>
<version>${libpam4j.version}</version>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if the dependency is managed, the version should be ommitted.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants