Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used.
@ 2020-09-05  7:03 fosslinux
  2020-09-05 23:48 ` [PR REVIEW] " the-maldridge
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: fosslinux @ 2020-09-05  7:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 360 bytes --]

There is a new pull request by fosslinux against master on the void-packages repository

https://github.com/fosslinux/void-packages Manual-noarch
https://github.com/void-linux/void-packages/pull/24684

Manual.md: remove mentions of noarch, note it should not be used.


A patch file from https://github.com/void-linux/void-packages/pull/24684.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-Manual-noarch-24684.patch --]
[-- Type: text/x-diff, Size: 3203 bytes --]

From b15edf7e84b140c8bd66254e4ad7742e4c4a7f89 Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Thu, 27 Aug 2020 17:17:50 +1000
Subject: [PATCH] Manual.md: remove mentions of noarch, note it should not be
 used.

---
 Manual.md | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/Manual.md b/Manual.md
index 5502c6e26f8..c4d1878c8e6 100644
--- a/Manual.md
+++ b/Manual.md
@@ -711,8 +711,7 @@ used.
 - `fetch_cmd` Executable to be used to fetch URLs in `distfiles` during the `do_fetch` phase.
 
 - `archs` Whitespace separated list of architectures that a package can be
-built for, available architectures can be found under `common/cross-profiles`
-alongside the `noarch` value for packages that do not contain any machine code.
+built for, available architectures can be found under `common/cross-profiles`.
 In general, `archs` should only be set if the upstream software explicitly targets
 certain architectures or there is a compelling reason why the software should not be
 available on some supported architectures.
@@ -725,9 +724,8 @@ Examples:
 	archs="x86_64-musl ~*-musl"
 	# Default value (all arches)
 	archs="*"
-	# Packages that do not depend on architecture-specific objects
-	archs=noarch
 	```
+Do not use noarch any longer. It is being deprecated and to be removed soon.
 
 <a id="explain_depends"></a>
 #### About the many types of `depends` variable.
@@ -914,8 +912,8 @@ can be used to pass arguments during compilation. If your package does not make
 extensions consider using the `gem` build style instead.
 
 - `gem` For packages that are installed using gems from [RubyGems](https://rubygems.org/).
-The gem command can be overridden by `gem_cmd`. `archs` is set to `noarch` unconditionally
-and `distfiles` is set by the build style if the template does not do so. If your gem
+The gem command can be overridden by `gem_cmd`. 
+`distfiles` is set by the build style if the template does not do so. If your gem
 provides extensions which must be compiled consider using the `gemspec` build style instead.
 
 - `ruby-module` For packages that are ruby modules and are installable via `ruby install.rb`.
@@ -1451,7 +1449,7 @@ type used to split architecture independent, big(ger) or huge amounts
 of data from a package's main and architecture dependent part. It is up
 to you to decide, if a `-data` subpackage makes sense for your package.
 This type is common for games (graphics, sound and music), part libraries (CAD)
-or card material (maps). Data subpackages are almost always `archs=noarch`.
+or card material (maps).
 The main package must then have `depends="${pkgname}-data-${version}_${revision}"`,
 possibly in addition to other, non-automatic depends.
 
@@ -1589,7 +1587,6 @@ The following variables influence how Haskell packages are built:
 Font packages are very straightforward to write, they are always set with the
 following variables:
 
-- `archs=noarch`: Font packages don't install arch specific files.
 - `depends="font-util"`: because they are required for regenerating the font
 cache during the install/removal of the package
 - `font_dirs`: which should be set to the directory where the package

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
@ 2020-09-05 23:48 ` the-maldridge
  2020-09-05 23:48 ` the-maldridge
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: the-maldridge @ 2020-09-05 23:48 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 194 bytes --]

New review comment by the-maldridge on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r484001023

Comment:
Should probably nix 'any longer' as well.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
  2020-09-05 23:48 ` [PR REVIEW] " the-maldridge
@ 2020-09-05 23:48 ` the-maldridge
  2020-09-06  7:32 ` [PR PATCH] [Updated] " fosslinux
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: the-maldridge @ 2020-09-05 23:48 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 267 bytes --]

New review comment by the-maldridge on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r484001021

Comment:
Deprecation is an atomic action, removal takes time.   So this should read "It is deprecated and is being removed"

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
  2020-09-05 23:48 ` [PR REVIEW] " the-maldridge
  2020-09-05 23:48 ` the-maldridge
@ 2020-09-06  7:32 ` fosslinux
  2020-09-06  7:33 ` [PR REVIEW] " fosslinux
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: fosslinux @ 2020-09-06  7:32 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 365 bytes --]

There is an updated pull request by fosslinux against master on the void-packages repository

https://github.com/fosslinux/void-packages Manual-noarch
https://github.com/void-linux/void-packages/pull/24684

Manual.md: remove mentions of noarch, note it should not be used.


A patch file from https://github.com/void-linux/void-packages/pull/24684.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-Manual-noarch-24684.patch --]
[-- Type: text/x-diff, Size: 3181 bytes --]

From 467dd416d9637bbdb8d20a8424a7ffd84148ac44 Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Thu, 27 Aug 2020 17:17:50 +1000
Subject: [PATCH] Manual.md: remove mentions of noarch, note it should not be
 used.

---
 Manual.md | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/Manual.md b/Manual.md
index 5502c6e26f8..4af3a7b78d9 100644
--- a/Manual.md
+++ b/Manual.md
@@ -711,8 +711,7 @@ used.
 - `fetch_cmd` Executable to be used to fetch URLs in `distfiles` during the `do_fetch` phase.
 
 - `archs` Whitespace separated list of architectures that a package can be
-built for, available architectures can be found under `common/cross-profiles`
-alongside the `noarch` value for packages that do not contain any machine code.
+built for, available architectures can be found under `common/cross-profiles`.
 In general, `archs` should only be set if the upstream software explicitly targets
 certain architectures or there is a compelling reason why the software should not be
 available on some supported architectures.
@@ -725,9 +724,8 @@ Examples:
 	archs="x86_64-musl ~*-musl"
 	# Default value (all arches)
 	archs="*"
-	# Packages that do not depend on architecture-specific objects
-	archs=noarch
 	```
+Do not use noarch. It is deprecated and being removed.
 
 <a id="explain_depends"></a>
 #### About the many types of `depends` variable.
@@ -914,8 +912,8 @@ can be used to pass arguments during compilation. If your package does not make
 extensions consider using the `gem` build style instead.
 
 - `gem` For packages that are installed using gems from [RubyGems](https://rubygems.org/).
-The gem command can be overridden by `gem_cmd`. `archs` is set to `noarch` unconditionally
-and `distfiles` is set by the build style if the template does not do so. If your gem
+The gem command can be overridden by `gem_cmd`. 
+`distfiles` is set by the build style if the template does not do so. If your gem
 provides extensions which must be compiled consider using the `gemspec` build style instead.
 
 - `ruby-module` For packages that are ruby modules and are installable via `ruby install.rb`.
@@ -1451,7 +1449,7 @@ type used to split architecture independent, big(ger) or huge amounts
 of data from a package's main and architecture dependent part. It is up
 to you to decide, if a `-data` subpackage makes sense for your package.
 This type is common for games (graphics, sound and music), part libraries (CAD)
-or card material (maps). Data subpackages are almost always `archs=noarch`.
+or card material (maps).
 The main package must then have `depends="${pkgname}-data-${version}_${revision}"`,
 possibly in addition to other, non-automatic depends.
 
@@ -1589,7 +1587,6 @@ The following variables influence how Haskell packages are built:
 Font packages are very straightforward to write, they are always set with the
 following variables:
 
-- `archs=noarch`: Font packages don't install arch specific files.
 - `depends="font-util"`: because they are required for regenerating the font
 cache during the install/removal of the package
 - `font_dirs`: which should be set to the directory where the package

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (2 preceding siblings ...)
  2020-09-06  7:32 ` [PR PATCH] [Updated] " fosslinux
@ 2020-09-06  7:33 ` fosslinux
  2020-09-06 13:29 ` q66
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: fosslinux @ 2020-09-06  7:33 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 153 bytes --]

New review comment by fosslinux on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r484034566

Comment:
done

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (3 preceding siblings ...)
  2020-09-06  7:33 ` [PR REVIEW] " fosslinux
@ 2020-09-06 13:29 ` q66
  2020-09-08 16:51 ` Chocimier
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: q66 @ 2020-09-06 13:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 163 bytes --]

New review comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r484070376

Comment:
this breaks the list

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (4 preceding siblings ...)
  2020-09-06 13:29 ` q66
@ 2020-09-08 16:51 ` Chocimier
  2020-09-08 19:55 ` q66
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Chocimier @ 2020-09-08 16:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 604 bytes --]

New review comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r485062870

Comment:
What do you mean? github renders it as 
```
<li>
<p><code>gem</code> For packages that are installed using gems from <a href="https://rubygems.org/" rel="nofollow">RubyGems</a>.
The gem command can be overridden by <code>gem_cmd</code>.
<code>distfiles</code> is set by the build style if the template does not do so. If your gem
provides extensions which must be compiled consider using the <code>gemspec</code> build style instead.</p>
</li>
```

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (5 preceding siblings ...)
  2020-09-08 16:51 ` Chocimier
@ 2020-09-08 19:55 ` q66
  2020-09-13  5:19 ` fosslinux
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: q66 @ 2020-09-08 19:55 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 176 bytes --]

New review comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r485161348

Comment:
must have looked wrong, nevermind

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (6 preceding siblings ...)
  2020-09-08 19:55 ` q66
@ 2020-09-13  5:19 ` fosslinux
  2020-09-13  5:19 ` [PR REVIEW] " fosslinux
  2020-09-13 12:11 ` [PR PATCH] [Merged]: " q66
  9 siblings, 0 replies; 11+ messages in thread
From: fosslinux @ 2020-09-13  5:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 148 bytes --]

New comment by fosslinux on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#issuecomment-691616210

Comment:
ping?

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR REVIEW] Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (7 preceding siblings ...)
  2020-09-13  5:19 ` fosslinux
@ 2020-09-13  5:19 ` fosslinux
  2020-09-13 12:11 ` [PR PATCH] [Merged]: " q66
  9 siblings, 0 replies; 11+ messages in thread
From: fosslinux @ 2020-09-13  5:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 187 bytes --]

New review comment by fosslinux on void-packages repository

https://github.com/void-linux/void-packages/pull/24684#discussion_r487483036

Comment:
afaict, there is no issue, is this ok?

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Merged]: Manual.md: remove mentions of noarch, note it should not be used.
  2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
                   ` (8 preceding siblings ...)
  2020-09-13  5:19 ` [PR REVIEW] " fosslinux
@ 2020-09-13 12:11 ` q66
  9 siblings, 0 replies; 11+ messages in thread
From: q66 @ 2020-09-13 12:11 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 199 bytes --]

There's a merged pull request on the void-packages repository

Manual.md: remove mentions of noarch, note it should not be used.
https://github.com/void-linux/void-packages/pull/24684

Description:


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2020-09-13 12:11 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-05  7:03 [PR PATCH] Manual.md: remove mentions of noarch, note it should not be used fosslinux
2020-09-05 23:48 ` [PR REVIEW] " the-maldridge
2020-09-05 23:48 ` the-maldridge
2020-09-06  7:32 ` [PR PATCH] [Updated] " fosslinux
2020-09-06  7:33 ` [PR REVIEW] " fosslinux
2020-09-06 13:29 ` q66
2020-09-08 16:51 ` Chocimier
2020-09-08 19:55 ` q66
2020-09-13  5:19 ` fosslinux
2020-09-13  5:19 ` [PR REVIEW] " fosslinux
2020-09-13 12:11 ` [PR PATCH] [Merged]: " q66

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).