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

[Request] biome #2291

Closed
2 of 4 tasks
MenacingPerson opened this issue Feb 19, 2023 · 7 comments
Closed
2 of 4 tasks

[Request] biome #2291

MenacingPerson opened this issue Feb 19, 2023 · 7 comments
Labels
priority:low This package affects just a few users. request:new-pkg New package requested.

Comments

@MenacingPerson
Copy link

MenacingPerson commented Feb 19, 2023

Link to the package base(s) in the AUR

https://aur.archlinux.org/packages/rome
https://aur.archlinux.org/packages/biome

Utility this package has for you

LSP and formatter for many languages

Do you consider the package(s) to be useful for every Chaotic-AUR user?

No, but for a great amount.

Do you consider the package to be useful for feature testing/preview?

  • Yes

Have you tested if the package builds in a clean chroot?

  • Yes

Does the package's license allow redistributing it?

YES!

Have you searched the issues to ensure this request is unique?

  • YES!

Have you read the README to ensure this package is not banned?

  • YES!

More information

It takes a really long time to build so would be nice to have in a prebuilt form

@MenacingPerson MenacingPerson added the request:new-pkg New package requested. label Feb 19, 2023
@dr460nf1r3 dr460nf1r3 added the priority:low This package affects just a few users. label Feb 19, 2023
@Technetium1
Copy link
Member

Build fails in check: rome.log

@Technetium1 Technetium1 added waiting:upstream-fix Waiting for fix in AUR or in the project iself. bug:PKGBUILD Something is wrong in the AUR side. labels Feb 22, 2023
@Technetium1
Copy link
Member

This also fails to build on my machine in the same way.

@MenacingPerson
Copy link
Author

MenacingPerson commented Feb 23, 2023

Oh... same here. I'm gonna report this to the maintainer.

I swear it built before?

@Technetium1 Technetium1 removed waiting:upstream-fix Waiting for fix in AUR or in the project iself. bug:PKGBUILD Something is wrong in the AUR side. labels May 9, 2023
@alerque
Copy link

alerque commented May 31, 2023

This package has mostly worked since I started maintaining it in the AUR, but the last couple releases are failing checks again. I've reported the issue upstream.

@Technetium1 Technetium1 added the waiting:upstream-fix Waiting for fix in AUR or in the project iself. label May 31, 2023
@Technetium1 Technetium1 reopened this May 31, 2023
@xiota
Copy link
Contributor

xiota commented Sep 19, 2023

@alerque Different error now.

[Fails to build](error: could not compile `rome_formatter`)
   Compiling rome_control_flow v0.0.0 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_control_flow)
warning: variable does not need to be mutable
   --> crates/rome_control_flow/src/builder.rs:149:22
    |
149 |     pub fn with_node(mut self, node: impl Into<SyntaxElement<L>>) -> Self {
    |                      ----^^^^
    |                      |
    |                      help: remove this `mut`
    |
    = note: `#[warn(unused_mut)]` on by default

warning: `rome_control_flow` (lib) generated 1 warning (run `cargo fix --lib -p rome_control_flow` to apply 1 suggestion)
   Compiling rome_json_factory v0.0.1 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_json_factory)
   Compiling rome_parser v0.0.1 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_parser)
   Compiling rome_js_syntax v0.0.2 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_js_syntax)
   Compiling rome_formatter v0.0.1 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_formatter)
   Compiling rome_fs v0.0.0 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_fs)
warning: ambiguous glob re-exports
 --> crates/rome_formatter/src/prelude.rs:1:9
  |
1 | pub use crate::builders::*;
  |         ^^^^^^^^^^^^^^^^^^ the name `BestFitting` in the type namespace is first re-exported here
2 | pub use crate::format_element::*;
  |         ------------------------ but the name `BestFitting` in the type namespace is also re-exported here
  |
  = note: `#[warn(ambiguous_glob_reexports)]` on by default

   Compiling rome_json_parser v0.0.1 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_json_parser)
   Compiling rome_deserialize v0.0.0 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_deserialize)
   Compiling rome_analyze v0.0.0 (/home/main-builder/pkgwork/src/tools-cli-v12.1.3/crates/rome_analyze)
error[E0080]: evaluation of constant value failed
   --> crates/rome_formatter/src/format_element.rs:386:1
    |
386 | static_assert!(std::mem::size_of::<crate::format_element::Tag>() == 16usize);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ attempt to divide `0_i32` by zero
    |
    = note: this error originates in the macro `static_assert` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0080]: evaluation of constant value failed
   --> crates/rome_formatter/src/format_element.rs:395:1
    |
395 | static_assert!(std::mem::size_of::<crate::FormatElement>() == 24usize);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ attempt to divide `0_i32` by zero
    |
    = note: this error originates in the macro `static_assert` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0080`.
warning: `rome_formatter` (lib) generated 1 warning
error: could not compile `rome_formatter` (lib) due to 2 previous errors; 1 warning emitted
warning: build failed, waiting for other jobs to finish...
==> ERROR: A failure occurred in build().

@alerque
Copy link

alerque commented Sep 19, 2023

That needs to be reported upstream, it is not a packaging issue.

@xiota
Copy link
Contributor

xiota commented Sep 19, 2023

https://github.com/rome/tools/

This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

They link to a new project:

https://github.com/biomejs/biome
https://biomejs.dev/blog/annoucing-biome

@xiota xiota changed the title [Request] rome [Request] biome Dec 20, 2023
xiota added a commit that referenced this issue Dec 20, 2023
@xiota xiota closed this as completed Dec 24, 2023
@xiota xiota reopened this Dec 24, 2023
@Technetium1 Technetium1 removed the waiting:upstream-fix Waiting for fix in AUR or in the project iself. label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low This package affects just a few users. request:new-pkg New package requested.
Development

No branches or pull requests

5 participants