-
Notifications
You must be signed in to change notification settings - Fork 21
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
Improve generated .ebuild
output
#132
Open
hololeap
wants to merge
14
commits into
gentoo-haskell:master
Choose a base branch
from
hololeap:improve-ebuild-output
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This data is fed into |
hololeap
force-pushed
the
improve-ebuild-output
branch
from
August 28, 2023 23:17
d31b93e
to
3481040
Compare
hololeap
force-pushed
the
improve-ebuild-output
branch
from
February 8, 2024 21:55
0d73e92
to
16e5b05
Compare
This will assist testing by allowing EBuild data to be read from a file Signed-off-by: hololeap <[email protected]>
Signed-off-by: hololeap <[email protected]>
Signed-off-by: hololeap <[email protected]>
Add "golden" tests which compare generated output to example ebuild files. This will help aid development of improvements for ebuild output. Signed-off-by: hololeap <[email protected]>
- Use an asterisk at the end of the version when possible - Add newline to DEPEND - Avoid backslashes by passing an array to src_configure - Drop trailing .0 components Thanks-to: Sam James <[email protected]> Signed-off-by: hololeap <[email protected]>
Some rendering utility functions and Prettyprinter re-exports Signed-off-by: hololeap <[email protected]>
Drop unused functions in Portage.Ebuild and associated test Signed-off-by: hololeap <[email protected]>
LinesBuilder is a custom Monad that works by building up a list of prettyprinter Docs and then concats them with vcat. It is helpful for when a logical piece of the ebuild may use a variable number of lines, including no lines at all (for instance when no CABAL_HACKAGE_REVISION or CABAL_PN are used). Improve some formatting Signed-off-by: hololeap <[email protected]>
Signed-off-by: hololeap <[email protected]>
hololeap
force-pushed
the
improve-ebuild-output
branch
from
March 5, 2024 03:07
37f5913
to
bcf12ee
Compare
049940
approved these changes
Aug 7, 2024
@049940 ❓ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a PR for generally improving the quality of generated
.ebuild
files. Example.ebuild
files have been added for "golden" testing. Moving forward, these will serve as guidelines or templates for how we want.ebuild
files to look.We can leverage the use of the
prettyprinter
package, which won't add much dependency footprint since it is also needed byoptparse-applicative
. This will be very useful for improving theshowEBuild
function (shown partially here).hackport/src/Portage/EBuild.hs
Lines 98 to 103 in df86a22
||
inbuild-depends
#127