Skip to content

jana2: backport perfetto#946

Draft
wdconinc wants to merge 3 commits into
developfrom
jana2-perfetto
Draft

jana2: backport perfetto#946
wdconinc wants to merge 3 commits into
developfrom
jana2-perfetto

Conversation

@wdconinc

@wdconinc wdconinc commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

This PR backports the JANA2 perfetto functionality to 2026.02.00. Screenshot in JeffersonLab/JANA2#497.

What is the urgency of this PR?

  • High (please describe reason below)
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue: backport perfetto)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

@wdconinc wdconinc marked this pull request as ready for review June 1, 2026 16:15
Copilot AI review requested due to automatic review settings June 1, 2026 16:15
## 🤖 Automated Style Fixes

This PR applies automated style fixes for #946.

### Changes Applied
- `spack style --fix` formatting
- Ensures compliance with Spack style guidelines

### Original PR
- **Number**: #946
- **Branch**: `jana2-perfetto`
- **Title**: jana2: backport perfetto

### How to Use
1. Review the changes in this PR
2. If acceptable, merge this PR instead of the original
3. Or, pull these changes into your original branch:
   ```bash
   git fetch origin style-fix-pr-946
   git merge origin/style-fix-pr-946
   ```

---
*This PR was automatically created by the [style-and-audit
workflow](https://github.com/eic/eic-spack/actions/runs/26767087507).*

Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR backports Perfetto tracing support into the jana2 Spack package for the 2026.02.00 release, exposing it as an optional build-time feature.

Changes:

  • Add a new +perfetto variant for JANA2 (intended for @2026.02.00).
  • Apply an upstream JANA2 patch that introduces Perfetto tracing support for @2026.02.00.
  • Plumb the variant into CMake via -DUSE_PERFETTO={ON,OFF}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread spack_repo/eic/packages/jana2/package.py
@wdconinc wdconinc enabled auto-merge (squash) June 1, 2026 16:25
@wdconinc wdconinc requested a review from a team June 1, 2026 16:25
@wdconinc wdconinc mentioned this pull request Jun 1, 2026
12 tasks
@wdconinc wdconinc removed the request for review from a team June 1, 2026 16:56
@wdconinc

wdconinc commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Patch does not apply cleanly to v2026.02.00 :-(

@wdconinc wdconinc marked this pull request as draft June 7, 2026 22:11
auto-merge was automatically disabled June 7, 2026 22:11

Pull request was converted to draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants