Skip to content

daemonize: fix build#476400

Merged
Sigmanificient merged 1 commit intoNixOS:masterfrom
starius:fix-daemonize
Jan 3, 2026
Merged

daemonize: fix build#476400
Sigmanificient merged 1 commit intoNixOS:masterfrom
starius:fix-daemonize

Conversation

@starius
Copy link
Copy Markdown
Contributor

@starius starius commented Jan 2, 2026

Include PR bmc/daemonize#39

It fixes daemonize after GCC update to 15.

In master and nixpkgs-unstable the package fails to build:

$ nix build .#daemonize                                                                                                                                                  
error: Cannot build '/nix/store/8m0xjz1hpc9fgqzas4dyd5dh5aly6zpi-daemonize-1.7.8.drv'.                                                                                                        
       Reason: builder failed with exit code 2.                                                                                                                                               
       Output paths:                                                                                                                                                                          
         /nix/store/ccnf5lgi8ai08b8a94kyqq3al7bng3fp-daemonize-1.7.8                                                                                                                          
       Last 25 log lines:                                                                      
       > getopt.c:52:16: error: too many arguments to function 'write'; expected 0, have 3     
       >    52 |         (void) write(2, s, (unsigned)strlen(s));\                             
       >       |                ^~~~~ ~                                                                                                                                                       
       > getopt.c:89:25: note: in expansion of macro 'ERR'                                     
       >    89 |                         ERR(": option requires an argument -- ", c);          
       >       |                         ^~~                                                   
       > getopt.c:48:20: note: declared here                                                   
       >    48 |         extern int write();\                                                  
       >       |                    ^~~~~                                                                                                                                                     
       > getopt.c:89:25: note: in expansion of macro 'ERR'                                     
       >    89 |                         ERR(": option requires an argument -- ", c);                                                                                                         
       >       |                         ^~~                                                   
       > getopt.c:53:16: error: too many arguments to function 'write'; expected 0, have 3                                                                                                    
       >    53 |         (void) write(2, errbuf, 2);}                                                                                                                                         
       >       |                ^~~~~ ~                                                                                                                                                       
       > getopt.c:89:25: note: in expansion of macro 'ERR'                                                                                                                                    
       >    89 |                         ERR(": option requires an argument -- ", c);          
       >       |                         ^~~                                                   
       > getopt.c:48:20: note: declared here                                                                                                                                                  
       >    48 |         extern int write();\                                                  
       >       |                    ^~~~~
       > getopt.c:89:25: note: in expansion of macro 'ERR'
       >    89 |                         ERR(": option requires an argument -- ", c);
       >       |                         ^~~
       > make: *** [Makefile:22: getopt.o] Error 1
       For full logs, run:
         nix log /nix/store/8m0xjz1hpc9fgqzas4dyd5dh5aly6zpi-daemonize-1.7.8.drv

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Jan 2, 2026
starius added a commit to starius/config that referenced this pull request Jan 2, 2026
Removed broken packages daemonize and netcat-openbsd.
There are PRs to add them back:
NixOS/nixpkgs#476197
NixOS/nixpkgs#476400
@Sigmanificient Sigmanificient linked an issue Jan 2, 2026 that may be closed by this pull request
@Sigmanificient Sigmanificient mentioned this pull request Jan 2, 2026
13 tasks
Include PR bmc/daemonize#39
It fixes daemonize after GCC update to 15.
Copy link
Copy Markdown
Member

@Sigmanificient Sigmanificient left a comment

Choose a reason for hiding this comment

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

Diff LGTM, clarified the patch name, and added links

@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Jan 2, 2026
@Sigmanificient
Copy link
Copy Markdown
Member

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 476400
Commit: 3276170e6b04df2bd7a1c2deb8914688f6e1e99c (subsequent changes)
Merge: 399347f149fecf4b480a640e115ffed242d6d5fb

Logs: https://github.com/Sigmanificient/nixpkgs-review-gha/actions/runs/20669107788


x86_64-linux

✅ 1 package built:
  • daemonize

aarch64-linux

✅ 1 package built:
  • daemonize

x86_64-darwin (sandbox = true)

✅ 1 package built:
  • daemonize

aarch64-darwin (sandbox = true)

✅ 1 package built:
  • daemonize

@Sigmanificient Sigmanificient added this pull request to the merge queue Jan 3, 2026
Merged via the queue into NixOS:master with commit a6f3568 Jan 3, 2026
27 checks passed
@starius starius deleted the fix-daemonize branch January 3, 2026 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants