Skip to content

devel/readline: fix build failure on DragonFly#1545

Merged
tuxillo merged 1 commit into
masterfrom
agentic/devel-readline-20260529-064610Z
May 30, 2026
Merged

devel/readline: fix build failure on DragonFly#1545
tuxillo merged 1 commit into
masterfrom
agentic/devel-readline-20260529-064610Z

Conversation

@dragonflybot

Copy link
Copy Markdown
Member

Generated by the DragonFly agentic build-fix loop and verified in a clean dev-env. Reviewed and accepted by operator operator before submission.

Problem

The build failed on @2026Q2 — classified compile-error, confidence high.

The DeltaPorts overlay patch dragonfly/patch-terminal.c is malformed: the line adding support for the cons25 terminal type has a double + (++ dumbterm = ...), causing the first + to leak from the unified-diff format into the source code as a literal unary-plus operator. The compiler then rejects + dumbterm = ... as a non‑lvalue assignment.

Evidence:

  • Compile error from terminal.c:
    ./terminal.c: In function '_rl_init_terminal_io':
    ./terminal.c:583:13: error: lvalue required as left operand of assignment
     +  dumbterm = STREQ (term, "dumb") || STREQ (term, "vt52") || STREQ (term, "adm3a") || STREQ (term, "cons25");
               ^
    
    The leading + before “dumbterm” is a stray diff‑addition marker, not part of the intended code.

Fix

Automated fix for the build failure in devel/readline.

What changed

1 file changed, +1/-2

  • ports/devel/readline/dragonfly/patch-terminal.c

Verification

Built successfully in a DragonFly dev-env (dsynth) for target @2026Q2.
Verified 2026-05-30T22:47:24.849883+00:00.

Provenance

  • Operator: operator
  • Agent: model=deepseek/deepseek-v4-pro attempts=1 tokens=457701

> Generated by the DragonFly agentic build-fix loop and verified in a clean dev-env. Reviewed and accepted by operator `operator` before submission.

## Problem

The build failed on `@2026Q2` — classified `compile-error`, confidence `high`.

The DeltaPorts overlay patch `dragonfly/patch-terminal.c` is malformed: the line adding support for the `cons25` terminal type has a double `+` (`++  dumbterm = ...`), causing the first `+` to leak from the unified-diff format into the source code as a literal unary-plus operator.  The compiler then rejects `+  dumbterm = ...` as a non‑lvalue assignment.

**Evidence:**

- Compile error from `terminal.c`:
  ```
  ./terminal.c: In function '_rl_init_terminal_io':
  ./terminal.c:583:13: error: lvalue required as left operand of assignment
   +  dumbterm = STREQ (term, "dumb") || STREQ (term, "vt52") || STREQ (term, "adm3a") || STREQ (term, "cons25");
             ^
  ```
  The leading `+` before “dumbterm” is a stray diff‑addition marker, not part of the intended code.

## Fix

Automated fix for the build failure in `devel/readline`.

## What changed

1 file changed, +1/-2

- `ports/devel/readline/dragonfly/patch-terminal.c`

## Verification

Built successfully in a DragonFly dev-env (dsynth) for target `@2026Q2`.
Verified 2026-05-30T22:47:24.849883+00:00.

## Provenance

- Operator: operator
- Agent: model=deepseek/deepseek-v4-pro attempts=1 tokens=457701

Signed-off-by: Fred [bot] <github@dragonflybsd.org>
@tuxillo tuxillo marked this pull request as ready for review May 30, 2026 22:48
@tuxillo tuxillo merged commit c3aacfe into master May 30, 2026
1 of 2 checks passed
@tuxillo tuxillo deleted the agentic/devel-readline-20260529-064610Z branch June 1, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants