Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix -fobjc-arc with GCC #5632

Merged
merged 4 commits into from
Jul 30, 2023
Merged

Conversation

Coeur
Copy link
Collaborator

@Coeur Coeur commented Jun 17, 2023

Fix #5626.

Note: @barracuda156 to give it a try.

We'll need to cherry pick it for 4.0.x branch too.

@Coeur Coeur added this to the 4.0.x milestone Jun 17, 2023
@Coeur Coeur added the type:build Changes that affect the build system label Jun 17, 2023
@ckerr ckerr marked this pull request as draft June 18, 2023 20:14
@Coeur Coeur marked this pull request as ready for review June 19, 2023 21:37
libtransmission/CMakeLists.txt Outdated Show resolved Hide resolved
libtransmission/CMakeLists.txt Outdated Show resolved Hide resolved
@Coeur Coeur force-pushed the coeur/clang-for-objc-arc branch 3 times, most recently from cc692bb to 0da70b1 Compare July 22, 2023 22:04
@Coeur Coeur requested a review from mikedld July 22, 2023 22:10
@Coeur
Copy link
Collaborator Author

Coeur commented Jul 28, 2023

@mikedld @ckerr Hoping we can get the build fix approved?

@ckerr
Copy link
Member

ckerr commented Jul 28, 2023

@Coeur there are still two discussions (between you, @mikedld, and @nevack) that are marked as unresolved, which is why I haven't taken any action.

Copy link
Member

@mikedld mikedld left a comment

Choose a reason for hiding this comment

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

Note: @barracuda156 to give it a try.

@ckerr ckerr merged commit 84f9d01 into transmission:main Jul 30, 2023
21 checks passed
@ckerr
Copy link
Member

ckerr commented Jul 30, 2023

Notes: Fixed potential build issue when compiling on macOS with gcc.

ckerr pushed a commit that referenced this pull request Jul 30, 2023
@Coeur Coeur deleted the coeur/clang-for-objc-arc branch July 30, 2023 03:07
mtremer pushed a commit to ipfire/ipfire-2.x that referenced this pull request Aug 30, 2023
- Update from version 4.0.3 to 4.0.4
- Update of rootfile not required.
- Changelog
Transmission 4.0.4
	This is a bugfix-only release. Everyone's feedback on 4.0.x has been very helpful -- thanks for all the suggestions, bug reports, and pull requests!
What's New in 4.0.4
All Platforms
	* Fixed bug in sending torrent metadata to peers. ([#5460](transmission/transmission#5460))
	* Avoid unnecessary heap memory allocations. ([#5520](transmission/transmission#5520), [#5527](transmission/transmission#5527))
	* Fixed filename collision edge case when renaming files. ([#5563](transmission/transmission#5563))
	* Fixed locale errors that broke number rounding when displaying statistics, e.g. upload / download ratios. ([#5587](transmission/transmission#5587))
	* Always use a fixed-length key query in tracker announces. This isn't required by the [spec](https://www.bittorrent.org/beps/bep_0007.html), but some trackers rely on that fixed length because it's common practice by other BitTorrent clients. ([#5652](transmission/transmission#5652))
	* Fixed potential Windows crash when [getstdhandle()](https://learn.microsoft.com/en-us/windows/console/getstdhandle) returns `NULL`. ([#5675](transmission/transmission#5675))
	* Fixed `4.0.0` bug where the port numbers in LDP announces are sometimes malformed. ([#5825](transmission/transmission#5825))
	* Fixed a bug that prevented editing the query part of a tracker URL. ([#5871](transmission/transmission#5871))
	* Fixed a bug where Transmission may not announce LPD on its listening interface. ([#5896](transmission/transmission#5896))
	* Made small performance improvements in libtransmission. ([#5715](transmission/transmission#5715))
macOS Client
	* Updated code that had been using deprecated API. ([#5633](transmission/transmission#5633))
Qt Client
	* Fixed torrent name rendering when showing magnet links in compact view. ([#5491](transmission/transmission#5491))
	* Fixed bug that broke the "Move torrent file to trash" setting. ([#5505](transmission/transmission#5505))
	* Fixed Qt 6.4 deprecation warning. ([#5552](transmission/transmission#5552))
	* Fixed poor resolution of Qt application icon. ([#5570](transmission/transmission#5570))
GTK Client
	* Fixed missing 'Remove torrent' tooltip. ([#5777](transmission/transmission#5777))
Web Client
	* Don't show `null` as a tier name in the inspector's tier list. ([#5462](transmission/transmission#5462))
	* Fixed truncated play / pause icons. ([#5771](transmission/transmission#5771))
	* Fixed overflow when rendering peer lists and made speed indicators honor `prefers-color-scheme` media queries. ([#5814](transmission/transmission#5814))
	* Made the main menu accessible even on smaller displays. ([#5827](transmission/transmission#5827))
transmission-cli
	* Fixed "no such file or directory" warning when adding a magnet link. ([#5426](transmission/transmission#5426))
	* Fixed bug that caused the wrong decimal separator to be used in some locales. ([#5444](transmission/transmission#5444))
transmission-remote
	* Fixed display bug that failed to show some torrent labels. ([#5572](transmission/transmission#5572))
Everything Else
	* Ran all PNG files through lossless compressors to make them smaller. ([#5586](transmission/transmission#5586))
	* Fixed potential build issue when compiling on macOS with gcc. ([#5632](transmission/transmission#5632))

Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:regression type:build Changes that affect the build system
Development

Successfully merging this pull request may close these issues.

CMakeLists unconditionally force -fobjc-arc flag for Apple which breaks build with GCC
5 participants