CycloneDX / CycloneDX/cyclonedx-core-java

Also read `license-mapping-override.json` to allow overriding license mapping

Aperta
#682 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Java
Stelle
120
Fork
90
Merge medio
12h 43m
PR unite (30g)
18

Descrizione

Change the code for [resolveFuzzyMatching()](https://github.com/CycloneDX/cyclonedx-core-java/blob/8a4d83e05889373024c3ea7ef1f630b35394e7ae/src/main/java/org/cyclonedx/util/LicenseResolver.java#L160)

In addition to `license-mapping.json` also read `license-mapping-override.json` and use the values from this file with priority.

This will allow end users to define their own mapping.
You can easily override the classpath in [Maven](https://github.com/jakub-bochenski/example-cyclonedx-core-java/blob/master/pom.xml#L51-L61) or Gradle.

Right now I'm building my own fork of `cyclonedx-core-java`, but I'm only interested in changing this file.
Allowing users to just provide this file on the classpath would be much easier than having to rebuild the whole library.

ATM I could just copy the `license-mapping.json` and change it and provide on the classpath, but
- it easily gets out of sync with upstream
- classpath order can be problematic

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start in src/main/java/org/cyclonedx/util/LicenseResolver.java at resolveFuzzyMatching() and inspect how license-mapping.json is loaded. Add support for license-mapping-override.json with its values taking priority, then verify the project’s existing tests or test suite. Done means users can provide the override resource on the classpath without rebuilding the library.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.