[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #1 from Josh Stone  ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=14904177

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Igor Gnatenko  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ignate...@redhat.com
   Assignee|nob...@fedoraproject.org|ignate...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #2 from Josh Stone  ---
FWIW, here's a scratch build trying an armv7hl bootstrap:
http://koji.fedoraproject.org/koji/taskinfo?taskID=14908548

Even x86 took ~100min, so this might take a while.  Note also that arm is Tier
2 by upstream standards, so it's not as well tested as x86.  It should be build
ok, but we'll see whether it can pass the %check...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #3 from Josh Stone  ---
Nearly 6 hours later, armv7hl failed a few run-pass tests like so:

> error: linking with `cc` failed: exit code: 1
> note: "cc" "-Wl,--as-needed" "-Wl,-z,noexecstack" "-L" 
> "/builddir/build/BUILD/rustc-1.10.0/armv7-unknown-linux-gnueabihf/stage2/lib/rustlib/armv7-unknown-linux-gnueabihf/lib"
>  "armv7-unknown-linux-gnueabihf/test/run-pass/vec-macro-no-std.0.o" "-o" 
> "armv7-unknown-linux-gnueabihf/test/run-pass/vec-macro-no-std.stage2-armv7-unknown-linux-gnueabihf"
>  "-Wl,--gc-sections" "-pie" "-Wl,-O1" "-nodefaultlibs" "-L" 
> "armv7-unknown-linux-gnueabihf/test/run-pass/" "-L" 
> "armv7-unknown-linux-gnueabihf/test/run-pass/vec-macro-no-std.stage2-armv7-unknown-linux-gnueabihf.run-pass.libaux"
>  "-L" "armv7-unknown-linux-gnueabihf/rt" "-L" 
> "/builddir/build/BUILD/rustc-1.10.0/armv7-unknown-linux-gnueabihf/stage2/lib/rustlib/armv7-unknown-linux-gnueabihf/lib"
>  "-Wl,-Bstatic" "-Wl,-Bdynamic" "-L" 
> "/builddir/build/BUILD/rustc-1.10.0/armv7-unknown-linux-gnueabihf/stage2/lib/rustlib/armv7-unknown-linux-gnueabihf/lib"
>  "-l" "std-e8edd0fd" "-l" "dl" "-l" "pthread" "-l" "gcc_s" "-l" "c" "-l" "m" 
> "-l" "rt" "-l" "util" "-l" "compiler-rt"
> note: /usr/bin/ld: /usr/lib/libc_nonshared.a(elf-init.oS): undefined 
> reference to symbol '__aeabi_unwind_cpp_pr0@@GCC_3.5'
> /lib/libgcc_s.so.1: error adding symbols: DSO missing from command line
> collect2: error: ld returned 1 exit status
> error: aborting due to previous error

I think there's a c/gcc_s interdependency that's not correctly managed here. 
At least on x86_64, the default C link line has something like this:

> -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s 
> --no-as-needed

i.e. -lgcc_s is positioned twice as-needed, before and after -lc.

I will see if I can reproduce this locally and file an rust issue.


For now, I think we should leave rust.spec as i686/x86_64 only, citing Tier 1
status.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #4 from Igor Gnatenko  ---
in Exclusive arch replace i686 with %{ix86}

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #5 from Josh Stone  ---
Even if it doesn't really cover *all* x86?  We only have a bootstrap compiler
for i686 in particular.  Although i586 is another known target at the moment,
upstream only builds std for it, and it's not even listed in the support tiers
at all.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-18 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Josh Stone  changed:

   What|Removed |Added

 Blocks||1357749




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357749
[Bug 1357749] Review Request: cargo - Rust's package manager and build tool
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Josh Stone  changed:

   What|Removed |Added

 Blocks||1359763




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1359763
[Bug 1359763] Rust Compiler
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||rust



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Igor Gnatenko  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #6 from Igor Gnatenko  ---
* License file FiraSans-LICENSE.txt is not marked as %license
* Package has .a files
* %doc %{_docdir}/%{name}/html/
-> nothing actually owns %{_docdir}/%{name} for -doc subpackage
* %define bootstrap_base
https://static.rust-lang.org/dist/%{bootstrap_date}/rustc-%{bootstrap_channel}
use %global
* BuildRequires:  python
-> is it really needed like python? not python2 or python3?
* -doc subpackage must be noarch
* is it possible to move all lib*.so into for example, %{_libdir}/rust/?

Bundled libs, add to License tag and include licenses to %license:
* libbacktrace is BSD
* hoedown is ISC
* I think "or" should be replaced with "and" in License or rust is licensed on
one or second license?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #7 from Josh Stone  ---
(In reply to Igor Gnatenko from comment #6)
> * License file FiraSans-LICENSE.txt is not marked as %license

OK, will change.

> * Package has .a files

Yes, libcompiler-rt.a, and technically all the .rlib files are static archives
too.  I could move them all to a rust-static package, and then the base rust
package will need to require that anyway.  Is that preferable?

> * %doc %{_docdir}/%{name}/html/
> -> nothing actually owns %{_docdir}/%{name} for -doc subpackage

Ok, it can share ownership of that directory, right?

> * %define bootstrap_base
> https://static.rust-lang.org/dist/%{bootstrap_date}/rustc-
> %{bootstrap_channel}
> use %global

Sure, will change.

> * BuildRequires:  python
> -> is it really needed like python? not python2 or python3?

You're right, it needs python2.  I'll change it.

> * -doc subpackage must be noarch

How strong is that "must"?  The documentation can vary by architecture in small
ways.  For instance, `std::os::linux::raw::stat`[1] varies everywhere.  That
happens to be deprecated, but it's just one I know off-hand; there may be more.

[1] https://doc.rust-lang.org/stable/std/os/linux/raw/struct.stat.html

> * is it possible to move all lib*.so into for example, %{_libdir}/rust/?

Maybe so, but why?  They have unique hashes in the names, so they won't
collide.  Then we'd have to either add rpaths to rustc and rustdoc, or add this
path to ld.so.conf.d/ anyway.  So why move them?

> Bundled libs, add to License tag and include licenses to %license:
> * libbacktrace is BSD
> * hoedown is ISC

OK to both.

> * I think "or" should be replaced with "and" in License or rust is licensed
> on one or second license?

Rust's COPYRIGHT explicitly says it is "at your option", so I think "or" is
correct.  Then I guess the additions for bundled libraries will be "and", so we
need something like this?

  License: ASL 2.0 or MIT, and BSD and ISC

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #8 from Josh Stone  ---
New files:
https://fedorapeople.org/~jistone/review/rust/v2/rust.spec
https://fedorapeople.org/~jistone/review/rust/v2/rust-1.10.0-2.fc26.src.rpm

I updated the things I agreed to above.  I didn't yet do anything about
changing the packaging static files, making -doc noarch, or moving libraries to
a subdirectory.  Please see my questions about those.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #9 from Igor Gnatenko  ---
* Fix license tag as we discussed on IRC (add license files if they are there)
* Make docs noarch as discussed on IRC
* Build tests in parallel

Probably we should also consider adding Provides: rustc (with version, %_isa
and etc.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #10 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/rust

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #11 from Josh Stone  ---
(In reply to Jon Ciesla from comment #10)
> Package request has been approved:
> https://admin.fedoraproject.org/pkgdb/package/rpms/rust

Thanks!

Do I need to wait for a bootstrap exception to proceed?
https://fedorahosted.org/fpc/ticket/643

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Jon Ciesla  changed:

   What|Removed |Added

 CC||limburg...@gmail.com



--- Comment #12 from Jon Ciesla  ---
Yes.  However that should be soon, especially since there's (IIRC) an FPC
meeting tomorrow.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Peter Lemenkov  changed:

   What|Removed |Added

 CC||fabian.deut...@gmx.de



--- Comment #13 from Peter Lemenkov  ---
*** Bug 915043 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #14 from Josh Stone  ---
It appears FPC did not make a quorum today. :(

In the meantime, here's another update.  I made the changes for license text
and a noarch -doc package.  I tried parallel %check, and while that does help
them build, many of them execute in parallel threads, so my system was heavily
oversubscribed while that was running.  I think it's better to keep that a
serial make for now.

https://fedorapeople.org/~jistone/review/rust/v3/rust.spec
https://fedorapeople.org/~jistone/review/rust/v3/rust-1.10.0-3.fc26.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-07-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Demi Marie Obenour  changed:

   What|Removed |Added

 CC||demetrioben...@gmail.com



--- Comment #15 from Demi Marie Obenour  ---
Note that Rust statically links by default for good reason: there is absolutely
no binary compatibility, since generic Rust code is always compiled as part of
whatever code instantiates the generic.  So I would strongly recommend
dynamically linking the compiler (only) and statically linking everything else
– any changes to one component will require transitive recompilation of
dependencies anyway.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #16 from Josh Stone  ---
Yes, I agree, Demi.  If you look at the spec %install, you'll see that I remove
all of the *.so libraries from the installed rustlib/$target/ path, so folks
using this compiler should only be able to link to the static rlibs.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #17 from Demi Marie Obenour  ---
I think that the dynamic libraries should still be available, for people who
want to write code that uses plugins (such as the compiler itself, which is
dynamically linked).  But any Rust program in Fedora that does not use plugins
should be statically linked.

We should check to see if the Rust parts of Gecko can compile on stable Rust.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #18 from Josh Stone  ---
The libraries are still there in %{_libdir} for rustc and rustdoc only.  They
are explicitly filtered from from requires/provides so we don't "leak" ABI.

Is there any plugin functionality in stable rust?  I thought this still
required using feature tags, which implies nightly rust (or cheating the
bootstrap key).  I don't think the distro should try to support any unstable
features.

Not sure if it's what you mean, but I have confirmed that Firefox 48's mp4parse
does compile just fine with stable rustc 1.10.0.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #19 from Martin Stransky  ---
Folks, it's really good to see this in Fedora, Thanks!

We will use that for Fedora Firefox builds when it's ready. Also Rust is going
to be a base Firefox compiler, see
https://bugzilla.mozilla.org/show_bug.cgi?id=1284816

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #20 from Josh Stone  ---
Martin, I'm glad you want to use it!  Will it be ok for now if it's only
enabled on i686/x86_64?  Hopefully you can enable it conditionally.  I do
eventually want to expand to all Fedora archs, but upstream's Tier 1 is just
x86.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #21 from Tom Tromey  ---
I looked briefly at the spec file.

I'd like to reiterate my desire for rust-gdb to "go away".
In an integrated system like Fedora, it isn't necessary.
The core of it looks like:

PYTHONPATH="$PYTHONPATH:$GDB_PYTHON_MODULE_DIRECTORY" gdb \
  -d "$GDB_PYTHON_MODULE_DIRECTORY" \
  -iex "add-auto-load-safe-path $GDB_PYTHON_MODULE_DIRECTORY" \
  "$@"


Here I think each part can be better done through system integration:

* PYTHONPATH is better handled by putting the rust/gdb python code
  into a directory where gdb and/or python already looks
* -d doesn't seem to be needed at all
* -iex isn't needed if the chosen directory is already safe according
  to gdb

Perhaps some minor patch to gdb would be appropriate here.
In this setup the rust-gdb package could still exist (maybe just
to avoid having rust depend on gdb); and even rust-gdb could still
exist, but just as a symlink to gdb.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #22 from Martin Stransky  ---
(In reply to Josh Stone from comment #20)
> Martin, I'm glad you want to use it!  Will it be ok for now if it's only
> enabled on i686/x86_64?  Hopefully you can enable it conditionally.  I do
> eventually want to expand to all Fedora archs, but upstream's Tier 1 is just
> x86.

Sure, x86 is a good start here, it's not mandatory for Firefox build now.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #23 from Josh Stone  ---
@tromey There's no existing path that's already in both auto-load safe-path and
python sys.path.  I do seem to need "directory" (-d) too, or else it says:

  warning: Missing auto-load script at offset 0 in section .debug_gdb_scripts

I think I figured out how to set all three automatically, by adding a new file
like /etc/gdbinit.d/rust.gdb containing:

  directory /usr/share/rust/etc
  add-auto-load-safe-path /usr/share/rust/etc
  python sys.path.append('/usr/share/rust/etc')

Does this look OK to you?


Maybe it would also be better to move this to "/usr/share/gdb/rust", so any
future "etc"-like items don't infect gdb's paths.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #25 from Josh Stone  ---
FPC approved the bootstrap exception, and now the builds are complete!  Rawhide
is ready, and I will submit bodhi updates for F24 and F25 soon.

I ended up masking the %check result, because I kept getting spurious failures.
 Different builds saw EAGAIN in a few different places, and otherwise failed
with different values in tcp-stress.rs (see [1] and [2]).  Odd that %check
never had an issue before on copr or koji scratch.  For now, it runs as far as
possible with 'make -k' so check result will be logged, just not gating.
 [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=822325
 [2] https://github.com/rust-lang/rust/pull/33640

Then I also had to make -doc have an arch after all, because rpmdiff rejects
the whole build if they produce any noarch differences.

I haven't applied the gdb tweaks to the actual package yet, but it's on my TODO
list.  You're welcome to file a bug on that, especially if you want to pursue
any changes in gdb to help this too.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #26 from Fedora Update System  ---
rust-1.10.0-4.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-08169a6354

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Fabian Deutsch  changed:

   What|Removed |Added

 CC||fdeut...@redhat.com
 Blocks||1366555



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Igor Gnatenko  changed:

   What|Removed |Added

  Alias|rust|



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-15 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #27 from Fedora Update System  ---
rust-1.10.0-4.fc25 has been pushed to the Fedora 25 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-08169a6354

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #28 from Fedora Update System  ---
rust-1.10.0-4.fc25 has been pushed to the Fedora 25 stable repository. If
problems still persist, please make note of it in this bug report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1356907] Review Request: rust - The Rust Programming Language

2016-08-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-27 06:33:16



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org