Page MenuHomeFreeBSD

Skip the use of lxqt-l10n as master port for lxqt related l10n ports.
ClosedPublic

Authored by jsm on Jul 8 2018, 9:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 16 2024, 8:15 PM
Unknown Object (File)
Dec 8 2024, 4:20 AM
Unknown Object (File)
Nov 14 2024, 5:11 AM
Unknown Object (File)
Oct 16 2024, 12:37 PM
Unknown Object (File)
Oct 16 2024, 12:36 PM
Unknown Object (File)
Sep 27 2024, 3:33 PM
Unknown Object (File)
Sep 27 2024, 5:25 AM
Unknown Object (File)
Sep 25 2024, 5:53 AM
Subscribers

Details

Summary

Do not use x11/lxqt-l10n as master port, fix install path of qtermwidget-l10n

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 17917
Build 17674: arc lint + arc unit

Event Timeline

Does it pass portlint? Have you run it through poudriere?

Should have mentioned, you want portlint -C

In D16185#343205, @jsm wrote:

Great, thanks. Like portlint fix the DATADIR things if you can. If you can't, no problem. Also, can you post your proposed commit message? Thanks!

I do not think you can override datadir for qterminal and qterminal-l10n . My commit message would be:

Do not use x11/lxqt-l10n as a  master port. Fix install path of qtermwidget-l10n.

Fixes portlint fatal errors about wrongly used masterdir. 
Only functional change is in qtermwidget-l10n.

And i would list the PR 229413 229414 229417 229418 229419 229421.

In D16185#343301, @jsm wrote:

I do not think you can override datadir for qterminal and qterminal-l10n .

Ok, that's fine.

My commit message would be:

Do not use x11/lxqt-l10n as a  master port. Fix install path of qtermwidget-l10n.

Fixes portlint fatal errors about wrongly used masterdir. 
Only functional change is in qtermwidget-l10n.

And i would list the PR 229413 229414 229417 229418 229419 229421.

You would need to list each PR on a separate line and you need to mention the mentor approval as well.

And i would list the PR 229413 229414 229417 229418 229419 229421.

You would need to list each PR on a separate line and you need to mention the mentor approval as well.

Sure, but 7. Commit Log Messages
states:
Multiple PRs may be specified on one line, separated by commas or spaces is that wrong?
And should it be:
PR: #DDDD
PR: #DDDD
or PR:#DDDD
#DDDD

This revision was not accepted when it landed; it landed in state Needs Review.Jul 9 2018, 4:40 PM
This revision was automatically updated to reflect the committed changes.