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.

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

New package: photon-1.0.1 #54594

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

New package: photon-1.0.1 #54594

wants to merge 2 commits into from

Conversation

R1shy
Copy link

@R1shy R1shy commented Mar 6, 2025

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-GLIBC)

closes #54532

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

this is my first pull request tips/advice is appreciated

@classabbyamp classabbyamp added the new-package This PR adds a new package label Mar 6, 2025
@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

sorry for not writing commit messages it is a bad habit, I just deleted the unused vars and prefixed the pkgdestdir var with an underscore because it is from the go build_style :)

@elbachir-one
Copy link
Contributor

sorry for not writing commit messages it is a bad habit, I just deleted the unused vars and prefixed the pkgdestdir var with an underscore because it is from the go build_style :)

You need to put the version number after the package name in the commit message. Also, you don't need hostmakedepends because of the build style.

# Template file for 'photon'
pkgname=photon
version=v1.0.1
revision=1
build_style=go
go_import_path=git.sr.ht/~ghost08/photon
short_desc="RSS reader as light as a photon with terminal + sixel"
maintainer="Emma Kadlecek <[email protected]>"
license="GPL-3.0-or-later"
homepage="https://git.sr.ht/~ghost08/photon"
distfiles="https://git.sr.ht/~ghost08/photon/archive/v${version}.tar.gz"
checksum=507ffe1903227f3ffab4f0d92ddb3ec49db34b285f276707606e23fb9eafe3c7

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

Do you mean the pull request name or should I add it when I make a new commit

@elbachir-one
Copy link
Contributor

elbachir-one commented Mar 6, 2025

Do you mean the pull request name or should I add it when I make a new commit

This
image
It should be something like this: 'New package: photon-1.0.1'. Keep this in mind for next time, and update the template accordingly.

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

Ok will do

@R1shy R1shy changed the title New Package: Photon New package: photon-1.0.1 Mar 6, 2025
@elbachir-one
Copy link
Contributor

You need to rebase to squash it into a single commit. The template name should not include a version number; it should just be the package name, like this:

# Template file for 'photon'
pkgname=photon
version=v1.0.1
revision=1
build_style=go
go_import_path=git.sr.ht/~ghost08/photon
short_desc="RSS reader as light as a photon with terminal + sixel"
maintainer="Emma Kadlecek <[email protected]>"
license="GPL-3.0-or-later"
homepage="https://git.sr.ht/~ghost08/photon"
distfiles="https://git.sr.ht/~ghost08/photon/archive/v${version}.tar.gz"
checksum=507ffe1903227f3ffab4f0d92ddb3ec49db34b285f276707606e23fb9eafe3c7

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

I don't know how to rebase it can you explain

@elbachir-one
Copy link
Contributor

elbachir-one commented Mar 6, 2025

I don't know how to rebase it can you explain

Check this, it should help a bit git rebase

The (n) represents the number of commits you've made. Basically, you keep the first one and change the others from (pick) to (squash).

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

Ok seems easy enough

@elbachir-one
Copy link
Contributor

Ok seems easy enough

Here is a quick tutorial.

untitled.mp4

@R1shy
Copy link
Author

R1shy commented Mar 6, 2025

ok full disclosure I have been doing most of this from a chromebook using github workspaces and it seems rebasing is funky on workspaces so I will rebase when I get home from school in about 3 hours

@elbachir-one
Copy link
Contributor

ok full disclosure I have been doing most of this from a chromebook using github workspaces and it seems rebasing is funky on workspaces so I will rebase when I get home from school in about 3 hours

Yeah, okay. After rebasing, just do a force push. git -f push

@elbachir-one
Copy link
Contributor

There is no need for:

CFLAGS="-fPIE -pie"
LDFLAGS="-fPIE -pie"
_PKGDESTDIR=${wrksrc}/destdir

.
.

remove hostmakedepends

update template comment to match version

fix comment

.
@R1shy
Copy link
Author

R1shy commented Mar 8, 2025

BUMP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package request: Photon
3 participants