rarias/bscpkgs
rarias/bscpkgs
Archived
2
0
Fork 3
Code Issues 2 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Update paraver to 4.12.0 #6

Closed
abonerib wants to merge 0 commits from abonerib/bscpkgs:update-paraver into master
pull from: abonerib/bscpkgs:update-paraver
merge into: rarias:master
Conversation 5 Commits - Files Changed -
abonerib commented 2025-07-21 12:01:32 +02:00
Collaborator
Copy Link

Updated to 4.12.0 and fixed the build on nixpkgs-unstable where libxml2 no longer provides the deprecated m4 macro AM_PATH_XML2.

Any reason to use fetchGit over fetchFromGithub?

Things done

  • Update kernel.nix
  • Update default.nix
  • Add patch to prevent focus stealing on redraw
  • Fix dont-expand-colors.patch
    - [ ] Fix mouse-label.patch
    - [ ] Update wxparaver.nix unused?
    - [ ] Update release.nix unused?
Updated to 4.12.0 and fixed the build on `nixpkgs-unstable` where libxml2 no longer provides the deprecated m4 macro `AM_PATH_XML2`. Any reason to use `fetchGit` over `fetchFromGithub`? #### Things done - [x] Update `kernel.nix` - [x] Update `default.nix` - [x] Add patch to prevent focus stealing on redraw - [x] Fix `dont-expand-colors.patch` ~~- [ ] Fix `mouse-label.patch`~~ ~~- [ ] Update `wxparaver.nix` unused?~~ ~~- [ ] Update `release.nix` unused?~~
abonerib added 2 commits 2025-07-21 12:01:33 +02:00 
Adds a new patch to fix libxml2: the m4 AM_PATH_XML2 macro has been
deprecated and is no longer included in the latest nixpkgs unstable.
Upstream recommends using `PKG_CHECK_MODULES` instead.
https://github.com/bsc-performance-tools/wxparaver/issues/18
rarias commented 2025-07-21 12:19:12 +02:00
Owner
Copy Link

Updated to 4.12.0 and fixed the build on nixpkgs-unstable where libxml2 no longer provides the deprecated m4 macro AM_PATH_XML2.

Can we merge this after the upgrade then?

Any reason to use fetchGit over fetchFromGithub?

I don't think so, I forgot why I choose it.

Things done

  • Update kernel.nix
  • Update default.nix
  • Add patch to prevent focus stealing on redraw
  • Fix dont-expand-colors.patch
  • Fix mouse-label.patch

I wouldn't worry about the mouse patch, it had some problems as well.

  • Update wxparaver.nix unused?
  • Update release.nix unused?

May be good to leave the build from release derivation, as sometimes they do private tarballs with some fixes (rarely), so we would want to have it handy. But I don't think we need to update them for now.

> Updated to 4.12.0 and fixed the build on `nixpkgs-unstable` where libxml2 no longer provides the deprecated m4 macro `AM_PATH_XML2`. Can we merge this after the upgrade then? > Any reason to use `fetchGit` over `fetchFromGithub`? I don't think so, I forgot why I choose it. > #### Things done > - [x] Update `kernel.nix` > - [x] Update `default.nix` > - [x] Add patch to prevent focus stealing on redraw > - [x] Fix `dont-expand-colors.patch` > - [ ] Fix `mouse-label.patch` I wouldn't worry about the mouse patch, it had some problems as well. > - [ ] Update `wxparaver.nix` unused? > - [ ] Update `release.nix` unused? May be good to leave the build from release derivation, as sometimes they do private tarballs with some fixes (rarely), so we would want to have it handy. But I don't think we need to update them for now.
abonerib commented 2025-07-21 12:30:53 +02:00
Author
Collaborator
Copy Link

Can we merge this after the upgrade then?

Do you mean nixos-25.11 or #5?

> Can we merge this after the upgrade then? Do you mean `nixos-25.11` or #5?
rarias commented 2025-07-21 12:34:25 +02:00
Owner
Copy Link

Do you mean nixos-25.11 or #5?

I mean #5, which upgrades to nixos-25.05 in bscpkgs (we only updated jungle).

Edit: Paraver is failing in the upgrade, so we will need to fix it as well:

wxparaver> tutorialsdownload.cpp:539: error: 'io_service' is not a member of 'boost::asio'; did you mean 'use_service'?
wxparaver>   539 |     boost::asio::io_service io_service;
> Do you mean `nixos-25.11` or #5? I mean #5, which upgrades to nixos-25.05 in bscpkgs (we only updated jungle). Edit: Paraver is failing in the upgrade, so we will need to fix it as well: ``` wxparaver> tutorialsdownload.cpp:539: error: 'io_service' is not a member of 'boost::asio'; did you mean 'use_service'? wxparaver> 539 | boost::asio::io_service io_service; ```
abonerib changed title from WIP: Update paraver to 4.12.0 to Update paraver to 4.12.0 2025-07-21 12:39:38 +02:00
abonerib added 1 commit 2025-07-21 12:44:34 +02:00
abonerib added a new dependency 2025-07-21 12:46:17 +02:00
#5 Update to nixos 25.05 and OmpSs-2 2025-06
abonerib force-pushed update-paraver from 64289d7962 to f1dfaa8395 2025-07-21 14:50:17 +02:00 Compare
abonerib force-pushed update-paraver from f1dfaa8395 to 6813741c4d 2025-07-21 18:14:48 +02:00 Compare
rarias requested changes 2025-07-21 18:26:29 +02:00
rarias left a comment
Owner

Thanks!

Thanks!
pkgs/paraver/default.nix Outdated
@ -30,0 +35,4 @@
# Fix for boost >=1.87 (thanks to gamezelda)
(fetchpatch {
url = "https://aur.archlinux.org/cgit/aur.git/plain/wxparaver-tutorialsdownload-Fix-Boost-ASIO-1.87.0-removals.patch?h=wxparaver&id=b0dcd08c472536e0a1a3cc1dfbc4c77d9f5e0d47";
sha256 = "sha256-YgGLxqj+6SvFEpyKl0RlUHl2qBo7r65/kI3cy0OKFYE=";
rarias commented 2025-07-21 18:20:06 +02:00
Owner

Can we copy the patch locally? I'm not sure how reliable is their cgit.

Can we copy the patch locally? I'm not sure how reliable is their cgit.
👍 1
abonerib marked this conversation as resolved
pkgs/paraver/kernel.nix Outdated
@ -26,4 +28,4 @@
patches = [
# https://github.com/bsc-performance-tools/paraver-kernel/pull/11
./dont-expand-colors.patch
rarias commented 2025-07-21 18:23:30 +02:00
Owner

They have changed the way they expand colors now. I'm not sure if this patch continues to work as intended, or if it is needed. I would recommend leaving the patch file but not adding it to patches until I have more time to look into it.

They have changed the way they expand colors now. I'm not sure if this patch continues to work as intended, or if it is needed. I would recommend leaving the patch file but not adding it to `patches` until I have more time to look into it.
👍 1
abonerib marked this conversation as resolved
abonerib force-pushed update-paraver from 6813741c4d to 7b0433320a 2025-07-21 18:34:29 +02:00 Compare
rarias force-pushed update-paraver from 7b0433320a to 1666c14a35 2025-07-22 13:00:44 +02:00 Compare
rarias commented 2025-07-22 14:12:51 +02:00
Owner
Copy Link

Merged in 1666c14a35

Merged in https://jungle.bsc.es/git/rarias/bscpkgs/commit/1666c14a35e08fd2eddcc206d622494435595fdc
rarias closed this pull request 2025-07-22 14:12:51 +02:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
rarias
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
abonerib arocanon rarias
No Assignees
2 Participants
Due Date
No due date set.
Depends on
#5 Update to nixos 25.05 and OmpSs-2 2025-06
rarias/bscpkgs
Reference: rarias/bscpkgs#6
No description provided.
Delete Branch "abonerib/bscpkgs:update-paraver"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?