Skip to content

Conversation

@Tang-Kin-Long
Copy link

@Tang-Kin-Long Tang-Kin-Long commented Dec 4, 2025

Hi, I am the contributor from RecLicense project of osslab-pku, taking on the issue "Contribute updates back to ScanCode?".
I observe that in Mulan License family, the rule for detecting Mulan Public License, Version 1 (Mulan PubL v1) is not existed.
I've done some tests in local environment to show that existing rules for Mulan PSL v1, ,Mulan PSL v2, Mulan PubL v2 can adequately detect and distinguish Mulan license.
Therefore, currently I only consider to add a new rule mulanpubl-1.0_1.RULE for Mulan PubL v1, which takes the content of mulanpubl-2.0_1.RULE as reference.

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Signed-off-by: Jianlang Deng csgarytang@gmail.com

Signed-off-by: Jianlang Deng <csgarytang@gmail.com>
@minghuizhou
Copy link

Still an open issue. @alex @aslakhellesoy

@pombredanne
Copy link
Member

@Tang-Kin-Long Thanks! If you see more examples of these notices that would be great!

Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

Thanks++ @Tang-Kin-Long Looks good.

Could you also add these required phrases on the rule text so we don't detect similar texts on a partial match from this license rule?

I would also make sense to add another license reference rule with http://license.coscl.org.cn/MulanPubL-1.0 as the rule text.

Tang-Kin-Long and others added 2 commits December 18, 2025 20:26
add required phrases "{{Mulan PubL v1}}" on rule text

Co-authored-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
add required phrases "{{http://license.coscl.org.cn/MulanPubL-1.0}}" on rule text
Signed-off-by: Jianlang Deng <csgarytang@gmail.com>

Co-authored-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
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.

4 participants