Skip to content

lynx: update to 2.9.3 - #30292

Open
LLE8 wants to merge 2 commits into
openwrt:masterfrom
LLE8:lynx_293
Open

lynx: update to 2.9.3#30292
LLE8 wants to merge 2 commits into
openwrt:masterfrom
LLE8:lynx_293

Conversation

@LLE8

@LLE8 LLE8 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

📦 Package Details

Maintainer: @LLE8

Description: Update version to 2.9.3, update source URLs, add list of config files


🧪 Run Testing Details

  • OpenWrt Version: SNAPSHOT, r35836-20d94d5a2b
  • OpenWrt Target/Subtarget: MediaTek Ralink MIPS / MT76x8
  • OpenWrt Device: TP-Link TL-MR3020v3

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

This PR have no patches

LLE8 added 2 commits August 17, 2026 21:26
Changelog: https://lynx.invisible-island.net/lynx2.9.3/CHANGES.html#index-v2.9.3
Also update source URLs.

Signed-off-by: Leonid Esman <leonid.esman@gmail.com>
Add conffiles section as described at
https://openwrt.org/docs/guide-developer/packages#buildpackage_defines

Signed-off-by: Leonid Esman <leonid.esman@gmail.com>
@LLE8 LLE8 changed the title Lynx 293 lynx: update to 2.9.3 Aug 17, 2026
@LLE8
LLE8 marked this pull request as ready for review August 17, 2026 19:19
Comment thread net/lynx/Makefile
Comment on lines +17 to +20
# Below are additional source URLs that may not work reliably sometimes or
# for users in some countries
# PKG_SOURCE_URL:=https://invisible-island.net/archives/lynx/tarballs/
# PKG_SOURCE_URL:=https://ftp.icm.edu.pl/packages/lynx/tarballs/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This can be removed, right?

@LLE8 LLE8 Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I suggest keeping these as spares.
First link is the main link on Lynx project home page https://lynx.invisible-island.net/lynx2.9.3/index.html . I reported the loading problem to рage maintainer, hope something will be fixed. It may be possible to get some diagnostic information using Curl.
For second link default '--connect-timeout 5' in OpenWrt build system is too small, as far as I understand. There may be network or server problems. I'll try to report the problem to the https://ftp.icm.edu.pl/ administrator.

@openwrt-ai openwrt-ai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed 2 new commits. Both commit messages match their diffs; version bump correctly keeps PKG_RELEASE:=1. Two inline notes, neither of which I'd consider a merge blocker — the conffiles one is worth a look, the other is cosmetic. The commented-out spare mirrors are already covered in the existing thread, so I've left them alone.


Generated by Claude Code

Comment thread net/lynx/Makefile
ac_cv_path_NCURSES_CONFIG=""

define Package/lynx/conffiles
/etc/lynx.cfg

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/etc/lynx.lss is installed the same way as lynx.cfg ($(INSTALL_DATA) ./files/lynx.lss $(1)/etc/ at net/lynx/Makefile:75) and is just as user-modifiable — it is the colour style sheet, and users routinely edit it. Without a conffiles entry it will be silently overwritten on upgrade while lynx.cfg is preserved. Was the omission intentional?

Suggested change
/etc/lynx.cfg
/etc/lynx.cfg
/etc/lynx.lss

Generated by Claude Code

Comment thread net/lynx/Makefile
https://fossies.org/linux/www/
PKG_HASH:=7374b89936d991669e101f4e97f2c9592036e1e8cdaa7bafc259a77ab6fb07ce
https://ftp2.osuosl.org/pub/blfs/conglomeration/lynx/
# Below are additional source URLs that may not work reliably sometimes or

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: trailing whitespace after or.

Suggested change
# Below are additional source URLs that may not work reliably sometimes or
# Below are additional source URLs that may not work reliably sometimes or

Generated by Claude Code

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.

3 participants