Initial commit of chrono-date (dependency for waybar)
Package inclusion needed to support T9691
Details
Details
- Reviewers
JoshStrobl - Group Reviewers
Triage Team - Commits
- R5328:93f3236a0eb5: Initial commit of chrono-date
Successfully built against waybar in local repository
Diff Detail
Diff Detail
- Repository
- R5328 chrono-date
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
This is a follow up to an abandoned submission here: https://dev.getsol.us/D11360
chrono-date is required to build waybar, as mentioned here: https://dev.getsol.us/T9691
This is the first time I've packaged something, please let me know if anything should have been done differently. Thanks!
Comment Actions
Good start! Please provide the patch for waybar as well. You do not need to wait for this to be included in the repository in order to package it, you can follow https://getsol.us/articles/packaging/local-repository/en/
You can use the child / parent revision functionality to have waybar depend on this patch, so this patch gets reviewed and landed first. Alternatively you can do Depends on D11783 in the summary.
Lastly, please provider a file called MAINTAINERS.md that has the following contents, replacing the applicable parts:
This file is used to indicate responsibility for the maintenance of this package. Individuals on this list should be the sole modifiers of the package, excluding cases where the Solus Team may need to perform necessary rebuilds, upgrades, or security fixes. This list should not be used for any direct contact usage. If you believe this package requires a package update, follow documentation from https://getsol.us/articles/packaging/request-a-package-update/en/. In the event this package no longer becomes sufficiently maintained, Core Team reserves the right to request a new maintainer or remove this package from the repository. - NAME - IRC: REPLACE_ME - Email: REPLACE_ME
| package.yml | ||
|---|---|---|
| 9 | Let's replace <chrono> with just chrono | |
| 11 | Let's replace <chrono> with just chrono | |
| pspec_x86_64.xml | ||
| 17 | All of this should be pattered into the main package as it is a devel only library, there's no linking involved, etc. So should be: patterns:
- /* | |
Comment Actions
I tried adding this through git/arc but getting a bit lost there.. can you accept the MAINTAINERS.md file from here?
| pspec_x86_64.xml | ||
|---|---|---|
| 17 | Not sure I understood this correctly, but I've added that line to package.yml | |
Comment Actions
Try git add MAINTAINERS.md, git commit --amend as normal then arc diff --edit to upload the updated patch to phabricator. Let me know if that helps
Comment Actions
first two steps seemed to work, and I can see in the git log the MAINTAINERS.md file has been added. No luck with 'arc diff --edit' though.
"Linting...
No lint engine configured for this project.
Running unit tests...
No unit test engine is configured for this project.
Usage Exception: No changes found. (Did you specify the wrong commit range?)"
Comment Actions
No luck with 'arc diff --edit' though.
Just run arc diff without --edit flag. A new promt will ask you for a commit message, and in this case you can write Add MAINTAINERS.md file.
arc diff --edit is used when you are going to update this patch message, in this case it is Initial commit of chrono-date. This is necessary if you had any typos there and you want to edit that commit message for command line.
Comment Actions
Appreciate all the help. Unfortunately, I get same error as above when using just 'arc diff' too. This might be overkill, but maybe easier if I just re-submit from scratch?
Comment Actions
This might be overkill, but maybe easier if I just re-submit from scratch
NO.
make clean To make sure that there aren't any eopkg file in the folder.
git add *
git commit --amend
arc diff --update D11783 To make sure that you are updating this diff.
Comment Actions
smh, MAINTAINERS.md added, other files removed? In a way, I feel that's progress.
Edit: are you able to see/use the other files in 'History', or do they all need to be updated under the same Diff? Gotta get to call it a night but will work more on this tomorrow. Thanks for the help and patience everyone!
Comment Actions
Yes it is strange that the other files are removed.
Are the other files, Makefile package.yml and pspec_x86_64.xml in the same directory with MAINTAINERS.md ?
| MAINTAINERS.md | ||
|---|---|---|
| 4 | While I am not 100% certain, I am pretty sure (h3o) is not your name. Adding your Phabricator name is not necessary, we know from the commit history. | |