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.

Sign up for GitHub

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

Jump to bottom

Collect more data from Maven POMs #505

Closed
licodeli opened this issue Feb 21, 2017 · 3 comments
Closed

Collect more data from Maven POMs #505

licodeli opened this issue Feb 21, 2017 · 3 comments
Labels
package scan

Comments

@licodeli
Copy link
Contributor

licodeli commented Feb 21, 2017 •
edited by pombredanne
Loading

"dependency_version", "dependency_scope", "contributor_email", "issue_url" fields are not captured yet in the Maven parser: and we should to support these fields
@pombredanne pombredanne changed the title Add more MAVEN2_FIELDS to parse POM, this is to enrich the function in minecode maven2 mapper Collect more data from Maven POMs Feb 21, 2017
licodeli pushed a commit that referenced this issue Feb 22, 2017
#505 Add more fields definition for maven in packagedcode
d2e6264
pombredanne pushed a commit that referenced this issue Feb 25, 2017
@pombredanne
#505 Add more fields definition for maven in packagedcode
28369e6
pombredanne pushed a commit that referenced this issue Feb 25, 2017
@pombredanne
#505 Add more fields definition for maven in packagedcode
2ad69e5
pombredanne pushed a commit that referenced this issue Feb 25, 2017
@pombredanne
#505 Add more fields definition for maven in packagedcode
9a6a33a
licodeli pushed a commit that referenced this issue Feb 28, 2017
#505: Remove extra spaces
1f3cea1
licodeli pushed a commit that referenced this issue Mar 1, 2017
#505 Add the structure to support parent and its sub-elements
5d4c17d
licodeli pushed a commit that referenced this issue Mar 1, 2017
#505 Add the dependency group elements support and add test cases
fcfc1fa
@pombredanne
Copy link
Member

@linexb Could you look instead in a completely different implementation of parsing Maven POMs? with https://github.com/sassoftware/pymaven and in a maven2.py script? I think this would be much better to get actual deps. And eventually we can patch pymaven and submit upstream our improvements as it may support dependencies well but not basic information fields.

See also sassoftware/pymaven#3

licodeli pushed a commit that referenced this issue Mar 1, 2017
#505: Refine the code format
09383b4
pombredanne pushed a commit that referenced this issue Apr 18, 2017
@pombredanne
Collect additional maven fields and update tests
060248b 
 * developer_organization
 * contributor_email
 * contributor_name 
 * contributor_organization 
 * distribution_management_repository_url

Link: #505
Signed-off-by: Li Ha <li_ha@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
pombredanne pushed a commit that referenced this issue Apr 18, 2017
@pombredanne
Add new structure to support Maven dependencies
64024b0 
 * support dependency and dependency group

Link: #505
Signed-off-by: Li Ha <li_ha@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
Copy link
Member

These are captured now but may not be full added to packages as part of #529

@pombredanne
Copy link
Member

This is merged in develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
package scan
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pombredanne @licodeli